Conversation
|
@sanjeetsuhag congrats on this! Please bump to me when it is ready for me to review. 😄 CC @lilleyse |
lilleyse
left a comment
lilleyse
left a comment
There was a problem hiding this comment.
The skeleton is good - just needs more detail here and there.
| @@ -0,0 +1,161 @@ | |||
| # 3DTILES_tile_metadata | |||
There was a problem hiding this comment.
Same comment about extras.draftVersion as https://github.com/CesiumGS/3d-tiles/pull/414/files#r435609626
| @@ -0,0 +1,161 @@ | |||
| # 3DTILES_tile_metadata | |||
There was a problem hiding this comment.
In the next iteration we should include support for arbitrary sized arrays instead of just VEC2, VEC3, VEC4, etc. Same goes for glTF EXT_3dtiles_feature_metadata. It seems like eventually everything's going to conform to the same schema.
| @@ -0,0 +1,161 @@ | |||
| # 3DTILES_tile_metadata | |||
There was a problem hiding this comment.
There should be padding rules like there are for batch table binary properties: https://github.com/CesiumGS/3d-tiles/tree/master/specification/TileFormats/BatchTable#padding
sanjeetsuhag
changed the title
|
Closing, but transferring most of the ideas here to a fresh PR. |
3 participants
This extension enables storage of per tile metadata. Specification is available here.