CesiumGS / cesium-unity

Bringing the 3D geospatial ecosystem to Unity
https://cesium.com/platform/cesium-for-unity/
Apache License 2.0
347 stars 83 forks source link

Refactor API for 3D Tiles 1.1 metadata support #389

Closed j9liu closed 7 months ago

j9liu commented 8 months ago

This PR refactors the plugin API to handle 3D Tiles 1.1 metadata implementation, which allows us to finally upgrade to the newest version of Cesium Native.

image

TODO:

Other:

j9liu commented 8 months ago

I think this PR can finally be reviewed! Though it is a meaty one 😬

j9liu commented 8 months ago

Thanks @kring ! I'll make the changes once you're done with the full review, so that I don't make you lose track of your current review. I also just opened https://github.com/CesiumGS/cesium-unity-samples/pull/65 in the Unity samples with changes based on the new API.

Also, you can test feature ID texture picking with San Francisco Ferry Building, asset 2333904. The script should work without needing to be changed.

j9liu commented 7 months ago

Thanks @kring ! I forgot to mention that I addressed all the comments and this is ready for another look 😅

kring commented 7 months ago

Looks great @j9liu! 🚀