search

CesiumGS / 3d-tiles

Specification for streaming massive heterogeneous 3D geospatial datasets :earth_americas:
2.04k stars 459 forks source link

Draft 1.1 external tileset class clarification #684

Closed javagl closed 2 years ago

javagl commented 2 years ago

This only adds the hint

Any class that is defined in the schema of an external tileset must also be defined in the top-level tileset schema.

in the main 'Metadata Schema' section, and thus, reflects the change from https://github.com/CesiumGS/cesium/pull/10387

If desired, the schema description could also be extended by

The class that property values conform to. The value must be a class ID declared in the classes dictionary of the metadata schema of the <right> tileset

Where <right> could be "containing" or "root/top-level".

There are some questions that could or should be investigated further, about namespacing or disambiguation (e.g. related to https://github.com/CesiumGS/cesium/issues/10085 ), but for now, this change is only an attempt to clarify the structure that is expected by CesiumJS

javagl commented 2 years ago

Note to myself: Don't forget to mirror this change in https://github.com/CesiumGS/3d-tiles/pull/682 ...

lilleyse commented 2 years ago

Thanks @javagl

javagl commented 2 years ago

Mirrored in https://github.com/javagl/3d-tiles/commit/492be698e03657197491448995c4b85836aa5686