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

Bloom doesn't account for camera viewport #7253

Closed
vultix opened this issue Jan 17, 2023 · 3 comments
Closed

Bloom doesn't account for camera viewport #7253

vultix opened this issue Jan 17, 2023 · 3 comments
Labels
C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled

Comments

@vultix
Copy link

vultix commented Jan 17, 2023

Bevy version

0.9.1

What you did

Tried applying a viewport to a camera with hdr and bloom turned on

What went wrong

The bloom doesn't account for the viewport correctly, leading to weird bloom artifacts.
image

Steps to Reproduce

Simply add a viewport to the Camera of the bloom example:

            camera: Camera {
                hdr: true, // 1. HDR must be enabled on the camera
                viewport: Some(Viewport {
                    physical_size: UVec2::new(600, 600),
                    ..default()
                }),
                ..default()
            },
@vultix vultix added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Jan 17, 2023
@JMS55
Copy link
Contributor

JMS55 commented Jan 17, 2023

This will be fixed in bevy 0.10 #6802.

@JMS55 JMS55 closed this as completed Jan 17, 2023
@vultix
Copy link
Author

vultix commented Jan 17, 2023

Ah, my apologies! I looked in the issues but didn't think to look in the PRs. Will do so from now on

@JMS55
Copy link
Contributor

JMS55 commented Jan 17, 2023

No worries, thanks for the bug report :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled
Projects
None yet
Development

No branches or pull requests

2 participants