 |
Revision: 7352 invalid value in lightgem gamma uniform |
 |
Discussion: http://forums.thedarkmod.com/topic/19239-soft-r-gamma/ |
|
|
I'm not sure what's left to do here. As far as I can tell, there are no visual issues and this feature is complete? |
|
|
Pending @stgatilov's review of the brightness formula. It's giving what looks like a Thief2 effect to me, rather than classic gamma boost. (And personally I like it more this way) |
 |
Currently, the soft gamma settings are complementary to the old gamma/brightness settings. They are controlled by two new cvars: r_ambientMinLevel: linearly transforms the colors so that everything which normally receives zero light will have color = minLevel. r_ambientGamma: has exactly the same mathemetical effect as the old r_gamma (but it affects only ambient lighting). More details here: http://forums.thedarkmod.com/topic/19239-soft-r-gamma/page-4#entry428380 P.S. Initially I thought that it would go through widespread testing and we will find out which settings we can drop. However, as soon as I made all settings configurable, the "soft gamma" topic lost all interest =) |
|
|
2.07 beta testers can offer final feedback for this. |
 |
Softgamma is very good. I have just one suggestion to make. The r_ambient_MinLevel range from 0.0 to 1.0. Wouldn't it make more sense to have the same type of scaling as regular lights in DR, i.e., an integer value between 0 and 255? |
|
|
IMHO the DR conventions should not apply to TDM cvars. We have e.g. r_gamma and r_lightscale both ranging 0-1 |
|
|
I see an opportunity for a little cleanup. http://wiki.thedarkmod.com/index.php?title=Advanced_TDM_Visuals_Tweaking Note that the r_postprocess cvars include gamma controlling shaders. We are probably duplicating operations between these and the new soft gamma options. |
|
|
While I haven't looked at the code, is the difference maybe that soft gamma is only applied to ambient light whereas the post-processing gamma and exposure (equivalent to soft gamma's linear correction) are applied globally? It sounds like that from the descriptions. Still, we have three places where gamma is applied now. At least one of them should be removed, probably r_gamma. |
|
|
I have changed target version to 2.08, as there is still some stuff to do and to discuss. See related issues. |
|
|
What's here to do ATM? Looks resolved to me. |
|
|
TODO: 1) Offer Soft Gamma in the GUI 2) Eliminate redundant Soft Gamma from the Postprocess shader Since these are sub-tasks, this tracker could be marked as closed. "Technically", it could've been marked closed in TDM 1.03 when JC Denton added soft gamma controls to the HDR-lite posprocess... Nobody was aware of the implications\potential at the time. |
|
|
There are legitimate reasons not to remove the gamma/brightness from post processing. It is the only way to adjust gamma/brightness without affecting the OS. I recommended to have the brightness and gamma sliders change the post-processing gamma and brightness as long as post-processing is enabled. If it is disabled, the regular method should be used and a warning displayed that these settings affect the OS. http://forums.thedarkmod.com/topic/19239-soft-r-gamma/?p=431978 |
|
|
Signing off on this for now |
 |
Will this get added in 2.08..? |
|
|
Here is the story. In svn rev 8573, a new "tonemap" shader is introduced. It applies gamma+brightness adjustment to the whole screen instead of color tables which did it previously. The color tables are of course disabled. As a side effect, gamma and brightness 1) no longer affect main menu (not nice), 2) no longer affect the whole OS (good), 3) now affect screenshot (supposedly good). Postprocessing did not live good with this new shader. Moreover, postprocessing does its own gamma+brightness adjustment, so it redundant. In svn rev 8581, I removed bloom completely and integrated all the other features of postprocessing into tonemap shader. So now there is only one shader with 6 cvars which perform final color adjustments (including gamma and brightness). The old postprocessing is removed completely. The cvars r_gamma nad r_brightness renamed to r_postprocess_gamma and r_postprocess_brightness to show that it is part of postprocessing, also set default gamma to 1.2 (not sure it is valid now, but probably a good starting point). In svn revs 15787 and 15788 updated gamma/brightness cvars in GUI and removed postprocess/bloom menu settings. Note that postprocessing can no longer be disabled, although setting the following cvars achieves the same effect: u_colorCurveBias 0 u_colorCorrectBias 0 |
|
|
I have created a related issue 0005163 for improving main menu in regards with postprocessing cvars. Aside from that, I guess this issue is done: this implementation will work for everyone in 2.08, taking their gamma/brightness values into account. |
- Anonymous
