View Issue Details

IDProjectCategoryView StatusLast Update
0004705The Dark ModGraphicspublic05.05.2019 17:42
Reporterduzenko Assigned To 
PrioritynormalSeveritynormalReproducibilityhave not tried
Status assignedResolutionopen 
Product VersionSVN 
Target VersionTDM 2.08Fixed in Version 
Summary0004705: Soft gamma
DescriptionReport related issues here
TagsNo tags attached.

Relationships

related to 0004160 closedduzenko Gamma controls effecting desktop, rather than engine. 
related to 0004545 acknowledged Gamma and brightness controls have no effect on external monitor 
related to 0004932 confirmed Graphical Settings Menu Overhaul 
child of 0003684 new Investigate GPL Renderer Improvements 

Activities

duzenko

duzenko

20.12.2017 12:16

developer   ~0009851

Revision: 7352
invalid value in lightgem gamma uniform
nbohr1more

nbohr1more

29.12.2017 20:12

developer   ~0009928

Discussion:

http://forums.thedarkmod.com/topic/19239-soft-r-gamma/
nbohr1more

nbohr1more

25.06.2018 06:29

developer   ~0010584

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?
duzenko

duzenko

25.06.2018 07:23

developer   ~0010587

Last edited: 25.06.2018 07:25

View 2 revisions

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)

stgatilov

stgatilov

10.10.2018 04:17

developer   ~0010785

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 =)
nbohr1more

nbohr1more

12.12.2018 05:05

developer   ~0010888

2.07 beta testers can offer final feedback for this.
STiFU

STiFU

22.12.2018 11:12

developer   ~0011047

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?
duzenko

duzenko

22.12.2018 13:19

developer   ~0011051

IMHO the DR conventions should not apply to TDM cvars. We have e.g. r_gamma and r_lightscale both ranging 0-1
nbohr1more

nbohr1more

27.12.2018 03:47

developer   ~0011143

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.
STiFU

STiFU

27.12.2018 10:05

developer   ~0011146

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.
STiFU

STiFU

30.12.2018 10:29

developer   ~0011191

I have changed target version to 2.08, as there is still some stuff to do and to discuss. See related issues.
duzenko

duzenko

16.02.2019 07:42

developer   ~0011605

What's here to do ATM? Looks resolved to me.
nbohr1more

nbohr1more

16.02.2019 09:50

developer   ~0011608

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.
STiFU

STiFU

17.02.2019 15:09

developer   ~0011613

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
duzenko

duzenko

05.05.2019 17:42

developer   ~0011785

Signing off on this for now

Issue History

Date Modified Username Field Change
20.12.2017 11:57 duzenko New Issue
20.12.2017 11:57 duzenko Assigned To => duzenko
20.12.2017 11:57 duzenko Status new => assigned
20.12.2017 12:16 duzenko Note Added: 0009851
29.12.2017 20:11 nbohr1more Relationship added related to 0004160
29.12.2017 20:12 nbohr1more Relationship added child of 0003684
29.12.2017 20:12 nbohr1more Note Added: 0009928
29.12.2017 20:13 nbohr1more Product Version => SVN
29.12.2017 20:13 nbohr1more Target Version => TDM 2.07
25.06.2018 06:28 nbohr1more Status assigned => feedback
25.06.2018 06:29 nbohr1more Note Added: 0010584
25.06.2018 07:23 duzenko Note Added: 0010587
25.06.2018 07:23 duzenko Status feedback => assigned
25.06.2018 07:25 duzenko Note Edited: 0010587 View Revisions
10.10.2018 04:17 stgatilov Note Added: 0010785
12.12.2018 05:05 nbohr1more Status assigned => feedback
12.12.2018 05:05 nbohr1more Note Added: 0010888
20.12.2018 16:54 nbohr1more Relationship added related to 0004545
22.12.2018 11:12 STiFU Note Added: 0011047
22.12.2018 13:19 duzenko Note Added: 0011051
22.12.2018 13:19 duzenko Status feedback => assigned
27.12.2018 03:47 nbohr1more Note Added: 0011143
27.12.2018 10:05 STiFU Note Added: 0011146
29.12.2018 08:51 STiFU Relationship added related to 0004932
30.12.2018 10:29 STiFU Note Added: 0011191
30.12.2018 10:29 STiFU Target Version TDM 2.07 => TDM 2.08
16.02.2019 07:42 duzenko Note Added: 0011605
16.02.2019 09:50 nbohr1more Note Added: 0011608
17.02.2019 15:09 STiFU Note Added: 0011613
05.05.2019 17:42 duzenko Note Added: 0011785
05.05.2019 17:42 duzenko Assigned To duzenko =>