-
Notifications
You must be signed in to change notification settings - Fork 6
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
Sky rendering is incorrect on mobile #25
Comments
Although I'm not familiar with As for the epsilon, is 1e-7 too small? Will something closer to 1e-6 (matching what common.glsl defines) resolve the issue? |
It must be - I'll try different values on my phone when I get a chance to see if anything fixes it. Or maybe another solution will come to mind. |
Just to add, I tried comparing the view Z instead of the depth value, but it didn’t work well for some reason I’m not sure. Maybe I need to revisit it. |
Turns out this isn't a depth issue at all 🤷 - see #26. Gotta love inconsistent behavior between platforms 😅 |
On my Samsung Galaxy S20 device - looks like the adjusted depth comparison from #3 (comment) is not so fool-proof 😬 Maybe there's a better way to handle this? Is there a reason
DepthTextures
aren't used here which might make this easier? Maybe that's anEffectComposer
limitation? I'll have to think through a solution more.The text was updated successfully, but these errors were encountered: