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

Atmosphere gets too bright with HDR turned on #12240

Open
jjspace opened this issue Oct 7, 2024 · 0 comments
Open

Atmosphere gets too bright with HDR turned on #12240

jjspace opened this issue Oct 7, 2024 · 0 comments

Comments

@jjspace
Copy link
Contributor

jjspace commented Oct 7, 2024

This is based off an observation from @ggetz while we were working on #12126

Atmosphere seems really bright when HDR is on. Check to make sure we're not accidentally doubly applying tonemapping for the atmosphere when HDR is on. Maybe verify if this issue was present with ACES tonemapping or not.

HDR off
2024-10-07_17-54

HDR on
2024-10-07_17-54_1

Notice that the atmosphere does get "brighter" when HDR is on. I notice it most as the blue part shrinks to almost nothing.

Another weird part of this is that if you crank the brightness way up you can see the total height of the atmosphere including the fully transparent pixels. That might be a separate bug but it also reveals that with HDR off those pixels are semi-transparent but with HDR on they become fully opaque.
This might hint at the actual issue here being that when HDR is on the transparency or light accumulation is not being calculated correctly

HDR off + brightness
2024-10-07_17-58

HDR on + brightness
2024-10-07_17-58_1

Reproduce

  1. Load the Atmosphere Sandcastle
  2. Turn off Fog and Ground atmosphere just so they don't distract, these do not seem to be impacted
  3. Move the camera to be looking at the horizon as shown in the pics above
  4. Toggle HDR under the Sky Atmosphere tab
  5. slide the Brightness Shift up higher to observe the "transparent vs opaque" oddity
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant