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

Ground primitives and textured entities on 3D Tiles #7434

Merged
merged 12 commits into from
Jan 2, 2019

Conversation

lilleyse
Copy link
Contributor

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.

on-3d-tiles

locahost demo

@cesium-concierge
Copy link

Thanks for the pull request @lilleyse!

  • ✔️ Signed CLA found.

Reviewers, don't forget to make sure that:

  • Cesium Viewer works.
  • Works in 2D/CV.
  • Works (or fails gracefully) in IE11.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.

🌍 🌎 🌏

@lilleyse
Copy link
Contributor Author

@likangning93 can you review the primitive changes and @bagnell the Scene/GlobeDepth changes?

Copy link
Contributor

@likangning93 likangning93 left a 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.

@pjcozzi
Copy link
Contributor

pjcozzi commented Dec 28, 2018

This + #7437 fixes #6636.

@pjcozzi
Copy link
Contributor

pjcozzi commented Dec 28, 2018

👍 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:
Copy link
Contributor

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.

@bagnell bagnell merged commit c1000bf into classification-fixes Jan 2, 2019
@lilleyse
Copy link
Contributor Author

lilleyse commented Jan 2, 2019

More CI timeouts. I restarted the build.

@lilleyse
Copy link
Contributor Author

lilleyse commented Jan 2, 2019

Well the previous commit passes, so I think this is fine. The last commit was CHANGES.md only.

@bagnell
Copy link
Contributor

bagnell commented Jan 2, 2019

Well the previous commit passes, so I think this is fine. The last commit was CHANGES.md only.

That's why I merged it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants