-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Artifacts in classification example in 1.70? #8953
Comments
I don't see an obvious place to start looking as the cause of this break in the CHANGES for 1.70, any advice @lilleyse ? |
Git bisect confirms this is the commit that introduced this issue: 2da52de |
Marking as priority next release since it's a regression that's breaking some people's applications. |
If we figure out why this regression happened, it might help us understand non-3D Tiles classification bugs on mobile. Last time I looked I didn't have any leads, but I'll look some more soon. |
Haven't had a chance to look into this enough, so it should be pushed to the next release (again). It's looking like a multi-frustum problem though. |
It's a multi-frustum problem that can be triggered by disabling log depth. |
This was reported over email - the classification example (https://sandcastle.cesium.com/index.html?src=3D%20Tiles%20Photogrammetry%20Classification.html) has these artifacts, almost as if the the classification tileset is being clipped too early by the near plane, only on mobile (Android or iOS).
I put together an example on Glitch to make it easier to test on mobile (you can fork it and change the CesiumJS include to 1.69 to see that it works there, and the artifacts are only present in 1.70.
https://glitch.com/edit/#!/classification-cesium-error?path=index.html%3A5%3A68
1.70
1.69
The text was updated successfully, but these errors were encountered: