#401 Update Chapter on Value-Only Serialization - first changes - Githubissues

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

#401 Update Chapter on Value-Only Serialization - first changes #403

Closed BirgitBoss closed 2 months ago

BirgitBoss commented 2 months ago

Chapter on Value-Only Serialization was updated:

update examples to provide Value-Only serialization (not in the context of a Submodel)
add table similar to metadata table: which attributes are displayed
add chapter for Submodel Example (with properties from SMT Technical Data, not Families any longer)
bugfix annotations instead of annoation in schema and examples
editorial bugfixes: update deprecated links

part of #401

BirgitBoss commented 2 months ago

@mjacoby @sebbader-sap please review (again). As discussed in todays Workstream AAS Meeting: here are the changes agreed to (see description of PR).

For any improvements of the chapter on the rules etc. a separate PR and issue would be needed: this just shows first improvements.

P.S. https://github.com/admin-shell-io/aas-specs/pull/372 from @sebbader w.r.t. changes of V3.0.2 of Part 1 was already accepted before

P.P.S. https://github.com/admin-shell-io/aas-specs/pull/375 was closed because I originally wanted to merge it to https://github.com/admin-shell-io/aas-specs/pull/372 but this does not make sense any longer. This PR can be looked at for alternative realizations of the Value-Only table and the rules

BirgitBoss commented 2 months ago

I fixed (1) and (2), thank you again for the careful reading.

The intention of the PR is not to solve all problems but to start with the changes we did have consensus on in the Workstream.

I copied your open topics to https://github.com/admin-shell-io/aas-specs/issues/401. It is a good idea to have checkboxes for this.

I expected https://github.com/admin-shell-io/aas-specs/pull/372 to solve (4) but I will check again

So Topics (3) to (8) are not covered in this PR. I updated the PR description: this PR will not close https://github.com/admin-shell-io/aas-specs/issues/401 but is just part of it.

BirgitBoss commented 2 months ago

I added an updated ChangeLog for the changes done

BirgitBoss commented 2 months ago

I will merge this PR, For additional findings please see https://github.com/admin-shell-io/aas-specs/pull/408