-
Notifications
You must be signed in to change notification settings - Fork 777
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 RTGI getting noisy for grazying angle geometry (case 1266462). #1771
Conversation
It appears that you made a non-draft PR! |
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.
Approved.
The noisiness at grazing angles is lower, as intended. See the comparison below:
I also found that with default settings the overall image is less blurry
Whats tested:
- If bug case is resolved (first comparison)
- If there was a regression in RTGI quality (second comparison). The quality of GI noticeably increased not only from grazing angles.
- If there is performance regression. On custom DXR office demo project, before and after PR I had the same ~27fps, on 720p.
- Checked all Screen Space Global Illumination parameters still work
- Verified that second denoise radius was removed
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.
Oh since we remove a parameter. We should update the documentation? Other than that it's good.
Doc updated |
https://fogbugz.unity3d.com/f/cases/1266462/
Purpose of this PR
This PR tries to improve the denoising of RTGI to get rid of high frequency noise at grazing angles. While there is no miracle solution. The changes in this PR should help reduce that artifacts.
This PR also removes the second denoiser radius that is now deduced from the first denoiser radius.
The PR should be tested extensively by the QA who reported the issue to see if this is overall an improvement.
Testing status
The DXR tests are green after updating the screenshots.
I tested mainly on the scene that was used to report the issue and after the fix I was not able to repro the high frequency noise.