-
Notifications
You must be signed in to change notification settings - Fork 472
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
Clarification for content availability of implicit tileset roots #769
Comments
I think this is referring to a tile without a {
"root": {
"boundingVolume": {
"region": [-1.318, 0.697, -1.319, 0.698, 0, 20]
},
"refine": "REPLACE",
"geometricError": 5000,
"implicitTiling": {
"subdivisionScheme": "QUADTREE",
"availableLevels": 21,
"subtreeLevels": 7,
"subtrees": {
"uri": "subtrees/{level}/{x}/{y}.json"
}
}
}
}
I think the line should be reworded to
|
Now this makes sense, on the level of the specification. It may raise questions about use-cases for an implicit tileset that does not have content at all. But one could make a case for such a tileset serving only as a purely spatial datastructure that stores metadata, for example. To avoid similar confusion, one could probably consider to elaborate that further, with something like
|
@javagl that sounds good. Could you open a PR with that wording? |
Yes, there are a few "smaller" things that I can try to address in a few small PRs during the weekend, like |
(Edited for more clarity...)
The specification text at https://github.com/CesiumGS/3d-tiles/tree/main/specification/ImplicitTiling#availability-1 says
I was a bit surprised by that statement: It is phrased as a requirement to omit this property, but it only refers to the root tile. But the information cannot be omitted when there is content available in other cells of the subtree.
It could be clarified that when the
contentAvailability
is omitted, it just meansavailableCount:0/constant:0
. But requiring to omit it depending on the presence of content only in the root tile does not seem to make sense.(This came up in the forum at https://community.cesium.com/t/omitting-content-availability-when-using-implicit-tiling/32238 )
The text was updated successfully, but these errors were encountered: