admin-shell-io / aas-specs

Repository of the Asset Administration Shell Specification IDTA-01001 - Metamodel
https://industrialdigitaltwin.org/en/content-hub/aasspecifications
Creative Commons Attribution 4.0 International
45 stars 26 forks source link

Optional values for all SubmodelElements #412

Open alexgordtop opened 2 months ago

alexgordtop commented 2 months ago

The constraints / cardinalities affecting the value of the SubmodelElements are inconsistent.

It is not obvious, why especially ReferenceElement.value, but also Property.value, File.value, Blob.value are optional, but e.g. RelationshipElement.first/second is required.

Request: Please revise required attributes with the aspect of defining a template structure / incomplete model in mind.


Issue caused by restriction: This mandatory constraint leads currently to issues with AASX serialization, since some IDTA SubmodelTemplates would like to specify the usage of e.g. RelationshipElements, but when deserializing a corresponding AASX, the parsers will fail according to a xml schema violation.


Beside SubmodelTemplates, there are other UseCases, where a kind of requested structure should be provided, that somebody else would have to fill like:

Use case: A component has an empty configuration, that the operator should fill to configure the asset. Using RelationshipElements is nearly impossible in the situation.

The same applies to File/Blob.contentType - if a component allows an attachment, it is forced to specify something like "/", if it is open for different kinds of values. From a modeler perspective, I would have preferred to not set the contentType in advance and describe in the description, that a kind of image or pdf is accepted as documentation, but no videos.

BirgitBoss commented 2 months ago

TF Metamodel 2024-04-17 Discussion not yet finalized but recommendation:

BirgitBoss commented 1 month ago

Decision Proposal TF Part 1 2024-05-15 Besides https://github.com/admin-shell-io/aas-specs/issues/412#issuecomment-2061377393 make the following attributes optional as well: File/contentType Blob/contentType

Reason:

BirgitBoss commented 2 weeks ago

Workstream AAS Sepc 2024-06-13 accepted make the following attributes optional