Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[panorama] compositing common scale selection #1394

Merged
merged 2 commits into from
Apr 12, 2023
Merged

Conversation

servantftechnicolor
Copy link
Contributor

Change the common scale selection for the various chunks.

I replaced min scale with max scale as the common scale for the laplacian compositing.

The Gaussian filter window used to be of size (1 + 2 * 2), with 2 the
radius of said Gaussian filter. With the change of the common scale from
the min scale to the max scale, it is updated to 1, but we might still
want to go back to the previous one later on.

The const variable gaussianFilterRadius is commented out since it is not
used anymore, but might be again in the future.
Copy link
Contributor

@cbentejac cbentejac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Tested on fisheye and roundshot panoramas and although evaluating the impact of this PR is not obvious, the final panoramas looked better with it.

@fabiencastan fabiencastan added this to the 3.1.0 milestone Apr 12, 2023
@fabiencastan fabiencastan merged commit fe8e68d into develop Apr 12, 2023
@fabiencastan fabiencastan deleted the dev/commonScale branch April 12, 2023 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants