Solari v7#24767
Conversation
|
You added a new example but didn't add metadata for it. Please update the root Cargo.toml file. |
stuartparmenter
left a comment
There was a problem hiding this comment.
The math looks right, but given the complexity and to double check it all I ran a small simulation against using your math to check for bias and it looks good after your fixes. I also wired it in to a number of example scenes (bistro, etc) and it looks better / don't see any visual regressions.
|
It looks like your PR has been selected for a highlight in the next release blog post, but you didn't provide a release note. Please review the instructions for writing release notes, then expand or revise the content in the release notes directory to showcase your changes. |
|
This is a good candidate for a dedicated release note. Feel free to just copy-paste the stuff from your PR description as the stub if you'd like :) |
This is a major rewrite of Solari.
The old version did DI, diffuse GI, and specular GI / mirror DI in 3 separate passes. Only diffuse rays were resampled, and no rays were traced for cross-domain MIS weight, leading to bias.
This new version is one unified algorithm that handles diffuse and specular DI and GI all in one reservoir. Resampling takes the full BRDF into account, as well as cross-domain visibility in MIS weights.
Advantages:
Disadvantages:
Showcase
Reference





This PR (with DLSS)
Main (with DLSS)
This PR
Main