Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

META-293 Fix multiple docs issues #81

Merged
merged 5 commits into from
Oct 2, 2024
Merged

Conversation

lucasotodegraeve
Copy link
Contributor

No description provided.

On class Role, change cardinality from `0..*` to `0..1` for creator,
contributor, publisher and actor.
Fix language tags on description, fix `sh:datatype` and bring shacl
and rdfs closer texts together.
Comment on lines 171 to 176
rdfs:label "cast members"@en ;
rdfs:label "castleden"@nl ;
rdfs:label "Membres de la distribution"@fr ;
rdfs:comment "Members of the cast."@en ;
rdfs:comment "Leden van de cast."@nl ;
rdfs:comment "Members of the cast."@fr ;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lucasotodegraeve Where possible, we try to remain as closely aligned as possible with the originel ontology that is available online, so in this case the EbuCore one.

We keep the complete ontologies that the data models rely on in the folder https://github.com/viaacode/datamodels/tree/main/external_ontologies_complete and keep the actual used subset in the folder https://github.com/viaacode/datamodels/tree/main/ontologies.

In this case specifically, I would keep the labels singular and just remove the random "Range: a string or a Cast"@en (since this is indeed wrong/strange, although this happens quite a lot with online ontologies).

Comment on lines 494 to 497
sh:or (
[ sh:datatype xsd:string ]
[ sh:datatype ebucore:Cast ]
);
Copy link
Contributor

@milvld milvld Oct 1, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lucasotodegraeve We don't use ebucore:Cast in the data model, so I would keep this as xsd:string.

The original ontologies and our data models don't necessarily have to align this closely. We use their properties and if possible what they suggest as a range, but we choose not to when applicable :) We keep the ontology as-is because it's not ours.

@milvld
Copy link
Contributor

milvld commented Oct 1, 2024

@lucasotodegraeve Back to you (:

Feel free to update the last modified dates in the headers of the relevant SHACL files (and perhaps add yourself to them as well ;) )

@lucasotodegraeve lucasotodegraeve merged commit 3bbd15e into main Oct 2, 2024
78 checks passed
@lucasotodegraeve lucasotodegraeve deleted the meta-293-fix-docs branch October 2, 2024 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants