-
Notifications
You must be signed in to change notification settings - Fork 471
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
Global feature IDs #265
Comments
I'm running into the same sorts of issue with classified point clouds. We may need a concept of global batch ids. |
There could be a case where you have a feature whose points are split between 8 tiles and you want to change the feature's color or edit its properties. A common case is changing its color on pick. In Cesium you would only be able to pick an eighth of the feature at a time. With a global id you could at least identify which features across all tiles belong to the same thing. This isn't really a problem though if your tileset is read-only and all you want to do is style it. |
Ah yes, we should scope out what it will take to address this and decide if it fits into 1.0. |
@lilleyse realistically can this be scoped to be small and quick for 1.0? Or hold for the future? |
This one can hold - it will require a bit of thinking to do properly and it's possible for an app to work around it. |
This is still very relevant for replacement-refinement tilesets that have the same features across tile boundaries or across LODs - e.g. photogrammetry and vector data. Global IDs can be stored in a property table indexed by local feature IDs. These global IDs can be used to access metadata from an external database, or maybe from an external property table (CC #136). If this is a common enough use case there could be a |
Related discussion in #407 about querying features in a tileset, possibly based on some global ID. |
Perhaps the existing |
@donmccurdy note that |
It sounded like the idea of a Global ID was to provide an ID that is not constrained to be an integer |
[0, featuresLength)
. The batch id should be allowed to be any arbitrary number.The text was updated successfully, but these errors were encountered: