-
-
Notifications
You must be signed in to change notification settings - Fork 157
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
HDR support #97
Comments
Odd. I have an HDR monitor and the brightness slider works fine. I have no idea what controls that "HDR/SDR" bar and I don't think it affects true HDR content. Only SDR. I would assume the registry controls it, but I can't find any documentation on where/how it's stored. |
This doesn't really help: https://support.microsoft.com/en-us/windows/hdr-and-wcg-color-settings-in-windows-10-2d767185-38ec-7fdc-6f97-bbc6c5ef24e6 |
I haven't found much on my own either. Without access to a bunch of HDR monitors, I don't know that I'll be able to do much about this. In a future update, I'll allow users to exclude their monitor from Twinkle Tray. So at least HDR displays can be excluded. If anyone can provide DDC/CI information about their own HDR display, documentation about adjusting HDR programmatically, or a PR for this feature, it would be greatly appreciated. |
@xanderfrangos I just dug through procmon when making the HDR toggle and can't find anything that stands out.. it's very chatty at about 47k lines in just a few seconds. If there's anything I can look for, LMK. Also, how can I provide you DDC/CI info? I've got two HDR compatible displays from different major manufacturers. |
relevant issue here: couldn't reset back to normal mode brightness after turning on & off HDR mode (not the windows fake one but the true HDR mode in a game or a movie), |
Also would like to see HDR support added if possible. Makes the app unusable because I like to keep Windows HDR enabled. |
I recently upgraded to Windows 11 and the brightness control of my HDR monitor stopped working. As soon as I turn off HDR, it works again. Even the v1.14-beta1 version does not allow the control that the stable version did on Windows 10. |
Ok I've found the registry key that the setting changes (tested on Win 11).
The value is from 1000 (0%) to 3500 (50%) to 6000 (100%) in dec. Once the I hope this is somewhat useful. |
@wywywywy That might actually help. Thanks! The only HDR display I have at the moment seems to control HDR brightness via DDC/CI, which is not normal. So someone else will have to look into this further for now. |
It appears that every 1000 equals 80 nit, although I kind of question that because my screen is supposed to be 400 nit max rather than 480 nit. Anyway I found out how to get the SDR balance through the Win32 API. I think that to be able to set the SDR balance we'll have to make a call to the display driver. Data structure: https://docs.microsoft.com/en-us/windows/win32/api/wingdi/ns-wingdi-displayconfig_sdr_white_level Get: https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-displayconfiggetdeviceinfo Set: https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-displayconfigsetdeviceinfo |
@xanderfrangos Brightness of SDR content can be controlled programatically (see my answer here), HDR brightness cant be controlled by "design". |
That's great! Thank you for sharing. I'm not sure when I'll have a chance to look into this again, but it's great to finally have a working example. |
For HDR brightness control, you could adjust the (reported) peak luminance in color profile, but that is a hacky solution at best and most HDR applications and games ignore that anyway and roll their own brightness calibration. I've also left some more info that may be relevant on the issue over at Monitorian, see link above. |
thats just your monitor. it depends on the specific monitor. For example on my samsung oddysey, I can control the screen brightness in the OSD menu when HDR is on |
IMO, this is super complicated and while it would be lovely to fix in the future. I don't think it would be worth the immediate effort. Dev time is precious. Let's consider the use-case scenarios. I mainly want HDR to operate how f.lux works [disable for full-screen apps] (in reverse). Enable HDR when full-screen apps are running. Disable when not (use brightness control). HDR would now auto-enable (and f.lux would auto-disable) at the same time when I launch a game or a movie in full-screen. This solves a few things:
My 2cents Thanks for awesome software! |
I don't agree. Implementing the HDR on-off switching properly and correctly is not simple either and i don't think it requires less resources to implement. Besides that, why would you want to limit HDR to fullscreen (games)? YT and other platforms can deliver HDR content just fine, i might want to watch some content in one window while doing other stuff in another. Just straight up dismissing windowed HDR content seems a little too far. Having what you propose is better than nothing, but it's only a workaround and doesn't really solve the original issue imo (not being able to control SDR brightness with HDR enabled). |
@xanderfrangos I am starting to work on adding HDR support (unless you are already doing that?). Let me know what you think about this. |
@lulle2007200 I'm not actively working on it. A PR for it would be very welcome. Are you working off the master or beta branch? In the beta branch I've added more code related to dynamically adding user-controlled sliders per-monitor. It might make sense to piggyback off of that instead of having it use separate logic/UI/settings for just the one slider. The HDR (well, SDR-in-HDR) slider could also replace the brightness slider and work off the existing "linked levels" and "normalize brightness" features. Either way, I'm sure the feature would make several users happy. If you can get the code for updating the SDR brightness working per-display, I can get it wired up to the UI if needed. |
Yeah, i am on the beta branch.
Im currently researching how to do that. If it is possible to set per monitor, it'd make sense to just replace the slider for HDR monitors. |
Hey, having this feature would be awesome, how is it coming along? Maybe I'll have a look at it too. In the meantime, I wanted to point out that there is a script here that is adjusting sdr 'brightness' over multiple monitors if that helps |
Some dev posted yesterday info about an undocumented Windows 10/11 API allowing to change SDR brightness on HDR display here: https://stackoverflow.com/a/78435051/1054589 |
When I switched to HDR monitors, I was shocked to find out that this hadn't been implemented yet :( |
When HDR support is enabled, brightness control on my monitor is disabled.
Instead, the following slider can control brightness of SDR apps, which acts like a 'pseudo' brightness bar.
Twinkle tray's brightness control in this scenario should either
Right now I can mess around with the brightness slier in Twinkle tray, but it looks like it's corrupting my monitors brightness setting (which are disabled) and has the ability to completely turn off my monitor at setting 0%, and in general doesn't look like it's doing the correct thing and possibly corrupting some state. I had to restart my monitor to reset settings back.
The text was updated successfully, but these errors were encountered: