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

Sky rendering is incorrect on mobile #25

Closed
gkjohnson opened this issue Dec 16, 2024 · 4 comments · Fixed by #26
Closed

Sky rendering is incorrect on mobile #25

gkjohnson opened this issue Dec 16, 2024 · 4 comments · Fixed by #26
Labels
package: atmosphere Specific to @takram/three-atmosphere

Comments

@gkjohnson
Copy link
Contributor

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 an EffectComposer limitation? I'll have to think through a solution more.

@shotamatsuda
Copy link
Member

shotamatsuda commented Dec 16, 2024

Although I'm not familiar with DepthTexture (after seeing the example), isn’t it overwritten during post-processing renders? It seems postprocessing is working around this in the pipeline redesign, which might address it.

As for the epsilon, is 1e-7 too small? Will something closer to 1e-6 (matching what common.glsl defines) resolve the issue?

@gkjohnson
Copy link
Contributor Author

As for the epsilon, is 1e-7 too small?

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.

@shotamatsuda
Copy link
Member

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.

@gkjohnson
Copy link
Contributor Author

gkjohnson commented Dec 16, 2024

Turns out this isn't a depth issue at all 🤷 - see #26. Gotta love inconsistent behavior between platforms 😅

@shotamatsuda shotamatsuda added the package: atmosphere Specific to @takram/three-atmosphere label Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: atmosphere Specific to @takram/three-atmosphere
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants