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

[BS] Time-Spans dont have a class #43

Closed
sennierer opened this issue Apr 28, 2023 · 3 comments
Closed

[BS] Time-Spans dont have a class #43

sennierer opened this issue Apr 28, 2023 · 3 comments
Assignees

Comments

@sennierer
Copy link
Member

I ran the preliminary shacl against the BS dataset.
Time-Spans dont seem to have a class:

Constraint Violation in ClassConstraintComponent (http://www.w3.org/ns/shacl#ClassConstraintComponent):
	Severity: sh:Violation
	Source Shape: ex:crm_P4_has_time-span
	Focus Node: <http://www.intavia.eu/bs/event/4559-24>
	Value Node: <http://www.intavia.eu/bs/timespan/22/4559>
	Result Path: crm:P4_has_time-span
	Message: Value does not have class crm:E52_Time-Span
@ptleskin
Copy link
Contributor

Ok, I noticed we have named the class crm:E52_Time_Span, I will change that to the correct one with a hyphen crm:E52_Time-Span

@sennierer
Copy link
Member Author

perfect! thanks!

@sennierer
Copy link
Member Author

thanks for updating that. Commited the new file to the already open PR in the source-data repo and the time-span violations are gone. Still remains the issue with the role_types, but that depends on where we add those triples obviously (added them to APIS for the moment and its valid now).

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

No branches or pull requests

3 participants