CesiumGS / cesium-unity

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

Fix incorrect ECEF->LLH transformation in CesiumCartographicPolygon. #482

Closed kring closed 3 months ago

kring commented 3 months ago

After #465, CesiumCartographicPolygon was incorrectly calling LongitudeLatitudeHeightToCenteredFixed when attempting to transform ECEF to Longitude/Latitude/Height. That is, it was transforming in the wrong direction. This made the vertex positions in the polygon incorrect, breaking the polygon functionality completely.

Reported here: https://community.cesium.com/t/cesiumpolygonrasteroverlay-can-not-clip-vectortilemap-in-v1-11/34093

kring commented 3 months ago

@csciguy8 can you please review and merge this one today so that we can include it in tomorrow's release?