imi-bigpicture / bigpicture-metaflex

Metadata schemas.
Apache License 2.0
7 stars 0 forks source link

Unify Dataset set element naming #60

Closed csc-jm closed 8 months ago

csc-jm commented 9 months ago

I was testing validation with these schemas and run into an issue with this: https://github.com/imi-bigpicture/Bigpicture-Metadata-Schema/blob/main/src/BP.dataset.xsd#L81-L85

I wonder what the reasoning is that the schema simultaneously allows using <BPDATASET> elements but then within a <DATASET_SET> it has to be named <DATASET> instead. Seems like it would be more sensible to use only one or the other.

erikogabrielsson commented 8 months ago

Yes, this is a typo. This PR fixes it https://github.com/imi-bigpicture/Bigpicture-Metadata-Schema/pull/61