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

CET Overlay is excessively bright in HDR #955

Open
Rool-IO opened this issue Jul 27, 2024 · 6 comments
Open

CET Overlay is excessively bright in HDR #955

Rool-IO opened this issue Jul 27, 2024 · 6 comments
Labels
bug Something isn't working

Comments

@Rool-IO
Copy link

Rool-IO commented Jul 27, 2024

Bug

Operating system: Windows 11
Game version: 2.12a (issue present with older versions as well)
CET version: Current (issue is a pre-existing one)
GPU: RTX 3080 FE

Description

CET overlay is drawn excessively bright in HDR.

Expected behaviour

CET overlay to be drawn at paper white brightness level

Reproduction

No idea. It just behaves this way on my install.

CET Overlay being overly bright in HDR.zip

@WSSDude WSSDude added the bug Something isn't working label Jul 28, 2024
@Rool-IO
Copy link
Author

Rool-IO commented Jan 11, 2025

@WSSDude , OptiScaler was having the same issue but Nitec has just fixed it. Here's hoping this is an easy one to put in CET...

@WSSDude
Copy link
Collaborator

WSSDude commented Jan 11, 2025

Seems to be using modified ImGui but, that should be applicable from the shader also I think looking at the change 🤔 And shader is part of the backend so shouldnt need ImGui modifications.

@dragonzkiller
Copy link
Contributor

I did look into this. Sure, it's tonemapped, but only changing the values ImGui actually sends to it's UI pipeline. We could do it at the shader level, but it's still only works on scRGB as it's a linear colorspace. PQ10 (metadata-less HDR10) is non-liner and would require more mapping.

We could take this as an opportunity to pull the shader out of dx12.cpp and implement HDR fixes. Maybe after #993 is finished?

@WSSDude
Copy link
Collaborator

WSSDude commented Jan 19, 2025

Yes, wanted to revisit this overall after #993 along with some other things.

@WSSDude
Copy link
Collaborator

WSSDude commented Jan 19, 2025

Spending most time doing stuff for RED4ext now, but it should simplify a lot of things down the line here.

@WSSDude
Copy link
Collaborator

WSSDude commented Feb 19, 2025

See #982 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants