-
Notifications
You must be signed in to change notification settings - Fork 470
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
Move CESIUM_RTC to the feature table for b3dm #263
Comments
Would this eliminate the need for the |
@lilleyse is it realistic to scope this for 1.0? E.g., can this be done in one day or less and is it easy for Cesium to preserve backwards compatibility for existing tilesets and exporters? |
This is in scope for 1.0. It seems like less than one day of effort and should not affect backwards compatibility with the This could also be a good item to add to the |
Resolved in #301 |
I think we should add Local_Geo_Center to b3dm, rather than RTC_CENTER, because the geo-position can be applied to different ellipsoid. |
In response to using other ellipsoids at runtime - the tiles would be positioned correctly but wouldn't the tile contents not fit properly? Either there would be intersections at the edges for smaller ellipsoids or gaps for larger ellipsoids. |
Now that b3dm tiles have a feature table, it would probably make sense for
RTC_CENTER
to exist in the feature table. It's still worth supportingCESIUM_RTC
as a glTF extension, but the feature tableRTC_CENTER
seems preferable from a spec point of view.The text was updated successfully, but these errors were encountered: