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

Validator - validate batch table hierarchy #122

Open
lilleyse opened this issue Aug 10, 2017 · 1 comment
Open

Validator - validate batch table hierarchy #122

lilleyse opened this issue Aug 10, 2017 · 1 comment

Comments

@lilleyse
Copy link
Contributor

@rms13

Spec: https://github.com/AnalyticalGraphicsInc/3d-tiles/tree/master/TileFormats/BatchTable#batch-table-hierarchy

Sandcastle demo in Cesium: http://cesiumjs.org/Cesium/Apps/Sandcastle/index.html?src=3D%20Tiles%20Batch%20Table%20Hierarchy.html&label=3D%20Tiles

This is one of the more complicated areas of the 3D Tiles spec, so let me know if you have questions. Also I welcome feedback on the spec writing itself if anything is unclear.

@rms13
Copy link
Contributor

rms13 commented Aug 17, 2017

  • instancesLength must be equal to sum of length property of all classes
  • instances.length for a class must be equal to its length property
  • classIds.length must be equal to instancesLength
  • parentIds.length must be equal to instancesLength when parentCounts is not defined
  • parentCount.length must be equal to instancesLength
  • parentIds.length must be equal to sum of parentCount values
  • classIds values should be 0-(classes.length- 1)
  • parentsIds values should be 0-(instancesLength- 1)
  • parentCounts values should be 0-(instancesLength- 1)
  • cyclic dependencies not allowed

@lilleyse
Am I missing anything?

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

No branches or pull requests

2 participants