-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
Solari: Improve specular GI #21907
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
Merged
Merged
Solari: Improve specular GI #21907
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
9779a0f
Solari: Improve specular GI
JMS55 00bbd6d
TODO
JMS55 f5a9fff
Merge branch 'main' into solari6-better-specular-gi
JMS55 4043b6c
Merge commit 'afae1c09bc92472c888a926aa7d5adf42f5e3fb2' into solari6-…
JMS55 847d613
Increase roughness threshold for diffuse GI reuse
JMS55 a90503c
Consts
JMS55 386f299
Proper emissive/NEE handling and MIS
JMS55 a27ffb8
Sharper mirror reflections
JMS55 9e9db20
Merge branch 'main' into solari6-better-specular-gi
JMS55 8b8b17d
Rename
JMS55 e66374f
Still not sure why this is needed
JMS55 8ff4ed4
Merge branch 'main' into solari6-better-specular-gi
JMS55 b87747e
Only apply MIS to NEE for emissive lights
JMS55 d9f5814
Skip emissive contribution on first specular GI bounce, since DI hand…
JMS55 bd8d9f3
Merge branch 'main' into solari6-better-specular-gi
JMS55 cedab96
Misc
JMS55 710db87
Fix NaNs when no lights exist in scene
JMS55 0218574
Merge branch 'main' into solari6-better-specular-gi
JMS55 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This threshold is below an input perceptual roughness of 0.63, which seems really high.
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.
It was based on a bunch of testing. 0.4 was the threshold where I didn't notice visual artifacts.
It'll eventually be configurable though.