-
-
Notifications
You must be signed in to change notification settings - Fork 212
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
SSAO color (or GTAO?) #211
Comments
Yes, that should be possible. I'll look into it when I get a chance. |
Added support for colorized ambient occlusion in |
Sandbox for reference: https://codesandbox.io/s/ball-pit-1-vlhu6 |
thanks, it looks awesome! the sandbox you posted had a halo for some reason, but i updated mine and it's exactly the effect i looked for https://codesandbox.io/s/ball-pit-1-mvkqs?file=/src/App.js |
@vanruesc from this question by @edankwan : https://twitter.com/edankwan/status/1280407226411290625 would that be possible, too? to deal out individual colors for objects? |
I don't know if that can be done with the current scalable ambient obscurance algorithm. I'll try adding light bounce based on @gkjohnson's GTAO implementation when I find some time. |
Opened a new issue for the light bounce feature: #214. |
@vanruesc @drcmda just note that without actual surface information (diffuse color, roughness, metalness) of the objects before shadows are applied bounce light won't really be able to "reveal" any real information in shadowed areas. When I was working on it I found that it wound up muddying an image in more complex cases because of this but in simple cases it didn't look too bad. Perhaps one day we'll have deferred rendering and we'll have more information for these effects. I think it would be a good option, though! The new AO color looks great, too. |
@gkjohnson Thanks for the heads-up 👍 |
this is what it should look like
with postprocessing: https://codesandbox.io/s/ball-pit-1-o5ir0
i know this isn't ground thuth AO but could we still set the AO color somehow? in this case i would like to make it red for instance.
The text was updated successfully, but these errors were encountered: