Skip to content
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

WIP Post process SSAO #3092

Closed
wants to merge 8 commits into from
Closed

WIP Post process SSAO #3092

wants to merge 8 commits into from

Conversation

raytranuk
Copy link
Contributor

@raytranuk raytranuk commented Apr 14, 2021

fixes: #1136

This PR includes a Post Process SSAO implementation based on and ported from Filament's excellent implementation.

image

NB: Future work may include applying SSAO more correctly only on ambient light

I confirm I have signed the Contributor License Agreement.

@raytranuk raytranuk self-assigned this Apr 14, 2021
@raytranuk raytranuk requested a review from a team April 15, 2021 16:42
@raytranuk raytranuk added area: graphics Graphics related issue enhancement labels Apr 15, 2021
@lostra01
Copy link

lostra01 commented Jun 1, 2021

@mvaligursky Looks great! Can you please help a tiny bit? I've grabbed the JS file, added it to PC, popped it onto my camera, and the effect does not seem to work at all.

Does it require any additional task to get it up and running?

@mvaligursky
Copy link
Contributor

That should be all needed, but I'm not 100% sure as I wasn't involved in this. It's possible an engine release was needed for this to work .. could you please try again now that the engine was released last week.

@mvaligursky
Copy link
Contributor

closing due to follow up PR: #3266

@willeastcott willeastcott deleted the PP-SSAO branch January 31, 2022 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: graphics Graphics related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Post-processed Ambient Occlusion
3 participants