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
Fixed LightProbes to have gamma correct when using gamma color space #1805
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good. I think this makes sense to land even if it changes existing projects, but we should land it in a new major version. That also means it shouldn't be backported.
Since this is a behavior change, can you add something to the "Changed" section in the change log? Keep the bug fix note in fixed as well though.
I agree with Peter, imo we can still land this for 10.x as is preview as long as we land for 10.1. Please add changed in the section as well and |
Added changed entry, will try to land into 10.1 if not will post-pone it to next major. |
…ght-probes-gamma-space
…universal/bugfix/light-probes-gamma-space
So as I understand we are still in 10.1.x preview, maybe we can land it? |
Please read;
PR Workflow for the Graphics repository:
Checklist for PR maker
need-backport-*
label. After you backport the PR, the label changes tobackported-*
.CHANGELOG.md
file.Purpose of this PR
Fix for case https://fogbugz.unity3d.com/f/cases/1268911/.
Testing status
Manual Tests: What did you do?
Automated Tests: Can not really add automation as we are currently testing linear color space only. Part of discussion is here https://unity.slack.com/archives/C89KFUUCT/p1596718321327600?thread_ts=1596617214.309600&cid=C89KFUUCT.
Yamato: https://yamato.prd.cds.internal.unity3d.com/jobs/902-Graphics/tree/universal%252Fbugfix%252Flight-probes-gamma-space
Test Project: Added in the case https://fogbugz.unity3d.com/f/cases/1268911/.
Linear
Gamma before fix
Gamma after fix
Comments to reviewers
As we always had incorrect LightProbe contribution in gamma space, this fix will change the look of current projects. Can we do something to make this easier for users?