Skip to content

#790 Fix Adjustment - 100% Gamma is too Colorful & Too Dark/Not Bright at all. #893

Merged
codeHusky merged 1 commit intosmartcmd:mainfrom
mutherbored:main
Mar 8, 2026
Merged

#790 Fix Adjustment - 100% Gamma is too Colorful & Too Dark/Not Bright at all. #893
codeHusky merged 1 commit intosmartcmd:mainfrom
mutherbored:main

Conversation

@eh-K
Copy link
Contributor

@eh-K eh-K commented Mar 8, 2026

Description

This PR slightly fixes what #790 achieved. The gamma value was nerfed too low and now 100% gamma is not bright at all as originally intended.

100% gamma was not bright at all and was instead the original sweet spot for between bright but colorful.

Changes

Increased the brightness impact of 0.5 to 1.2.
Lowered the <50% value from 0.5 to 0.4.

Previous Behavior

100% gamma was not bright at all and was instead the original sweet spot for between bright but colorful.

Root Cause

#790 lowered the slider value weight a bit too low.

New Behavior

Now 100% Gamma is the brightness you would expect but not insane like pre-fix. Not the grounded colorful amount.

Fix Implementation

Located the value modifier in GameRenderer and adjusted the brightness modifier from 0.5 to 1.2
I then lowered the <50% brightness modifier from 0.5 to 0.4.

AI Use Disclosure

No AI was used.

Related Issues

Screenshot 2026-03-07 193818

@eh-K eh-K changed the title Adjust gamma calculation based on slider value #790 Fix Adjustment - 100% Gamma is too Colorful & Not Dark/Bright Mar 8, 2026
@eh-K
Copy link
Contributor Author

eh-K commented Mar 8, 2026

This is 0% gamma.

Screenshot 2026-03-07 201624

@eh-K eh-K changed the title #790 Fix Adjustment - 100% Gamma is too Colorful & Not Dark/Bright #790 Fix Adjustment - 100% Gamma is too Colorful & Too Dark/Not Bright at all. Mar 8, 2026
@MotoLegacy
Copy link

Giving my not-asked-for OK on this as issue filer for #876

@codeHusky codeHusky merged commit a865cea into smartcmd:main Mar 8, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants