opengeospatial / cdb-volume-1

Repository for OGC CDB Volume 1
http://www.github.com/opengeospatial/cdb-volume-1
2 stars 2 forks source link

Added missing descriptor dataset to "Tiled T2DModel Datasets" table #15

Closed RobertRicci closed 11 months ago

RobertRicci commented 3 years ago

Add missing descriptor for T2D

ryanfranz commented 3 years ago

So what parts of the descriptor are missing? Maybe this was an oversight in CDB 3.2, but there is a Composite Material Table dataset (312). The only thing possibly missing is the list of textures and their available mipmap levels. Either way, we would prefer to not have to open multiple xml files to use the Tiled 2D dataset.

RobertRicci commented 3 years ago

The texture attributes (Texture Metadata from volume 6 - 6.14.5. Model Textures) was mainly of interest here. Texture resolution and mipmaps would still be relevant for T2D. I agree not all meta-data attributes are relevant, ideally we can choose only the attributes necessary for the context (ex. mass would be relevant to a Moving model, but not a T2D). I don't agree to change the vocation of the CMT for this. I understand adding this dataset can cause a combability and performance issue. On our side (CAE), we are able to work without it for now. I suggest we discard this change for 1.3 and consider it for a future version of the spec.

ryanfranz commented 3 years ago

To update my previous comment, I see the need now for this missing model descriptor file. The text change is ok with me

ryanfranz commented 1 year ago

@RobertRicci - I would recommend a few more changes for Volume 1:

I can help with these changes if you need