-
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
fix large entities on terrain/GroundPrimitives crossing IDL in 3D #6967
Conversation
Thanks for the pull request @likangning93!
Reviewers, don't forget to make sure that:
I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome. 🌍 🌎 🌏 |
2D/CV and IE11: as well as it did in |
@likangning93 what's going on here? This is from the example you posted. |
@hpinkos I think that's a result of Might need to discuss with @bagnell what we can do about this... adjusting the shadow volume height should help a bit in the meantime though. |
The problem with the screen shot is that the volumes are split but not closed. If you split the volume and close both on each side of the IDL, it should be fine. @likangning93 Can you add this? In GeometryPipeline, the triangles are split. Can you keep track of the new vertices added. Then you can project the vertices onto the plane of the IDL and triangulate. Then, add the new indices to the split geometry. |
Still haven't wrapped my head around why this is from a stencil perspective, but simulating "closed" volumes by hand-splitting the rectangle does seem to work as described on Sandcastle. @bagnell I'll open a separate issue and PR for this though. |
@likangning93 here's the issue for seeing that seam when ground primitives cross the IDL: #4746 |
This looks good. I'll merge. Don't forget about the other two PRs 😃 |
Part of #6951.
This doesn't fix the problem where ground primitives crossing the IDL are broken in columbus view and 2D, but that's going to be more complicated so I'll tackle that in a separate PR.
Give it a try here.