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

More flexible styling for classified point clouds #269

Closed
lilleyse opened this issue Nov 2, 2017 · 5 comments
Closed

More flexible styling for classified point clouds #269

lilleyse opened this issue Nov 2, 2017 · 5 comments

Comments

@lilleyse
Copy link
Contributor

lilleyse commented Nov 2, 2017

If a point cloud is classified, in that it uses batch ids and has a BATCH_LENGTH, the length of each batch table array is equal to BATCH_LENGTH rather than POINTS_LENGTH. This means it is not possible to support per-point attributes when the point cloud is classified. Styling based on a point's "intensity" or "height" isn't possible.

The change would require marking some batch table properties as per-batch properties and others as per-point properties.

@pjcozzi
Copy link
Contributor

pjcozzi commented Feb 18, 2018

Are you certain this would be part of a MVP 1.0?

If it doesn't affect basically all point cloud styling and can be added in a backwards-compatible way, it should be labeled next to reduce scope.

@lilleyse
Copy link
Contributor Author

It isn't required for 1.0. Any app can get around this limitation by storing classification as a per-point property, however it's more work to get the same functionality. Probably best to scope for next.

@lilleyse
Copy link
Contributor Author

See CesiumGS/glTF#1 - per-point and per-group properties can be specified together by using multiple batch tables.

@pjcozzi
Copy link
Contributor

pjcozzi commented Mar 8, 2021

CC CesiumGS/glTF#4

@lilleyse lilleyse removed the next label Nov 15, 2021
@ptrgags
Copy link
Contributor

ptrgags commented Nov 15, 2021

This is now possible with the EXT_mesh_features extension for glTF: KhronosGroup/glTF#2082. We allow multiple sets of feature ID attributes. See the Feature ID section of the spec

@ptrgags ptrgags closed this as completed Nov 15, 2021
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

3 participants