-
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
Ground primitives and textured entities on 3D Tiles #7434
Conversation
…on if there are no 3D Tiles commands
Thanks for the pull request @lilleyse!
Reviewers, don't forget to make sure that:
I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome. 🌍 🌎 🌏 |
@likangning93 can you review the primitive changes and @bagnell the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Primitive updates look good to me.
👍 at quick glance! |
CHANGES.md
Outdated
* Fixed an issue where classification primitives with the `CESIUM_3D_TILE` classification type would render on terrain. | ||
* Fixed an issue where 3D Tiles would show through the globe. | ||
|
||
##### Breaking Changes :mega: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Breaking changes should go first, before everything else.
More CI timeouts. I restarted the build. |
Well the previous commit passes, so I think this is fine. The last commit was CHANGES.md only. |
That's why I merged it. |
Follows up on #7422 and adds support for ground primitives and textured entities on 3D Tiles.
The default classification type is now
ClassificationType.BOTH
everywhere.locahost demo