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

5.1 DCAT Scope states 6 main classes, then shows 7. #1137

Closed
chris-little opened this issue Oct 24, 2019 · 5 comments
Closed

5.1 DCAT Scope states 6 main classes, then shows 7. #1137

chris-little opened this issue Oct 24, 2019 · 5 comments

Comments

@chris-little
Copy link

@chris-little chris-little commented Oct 24, 2019

Section "5.1 DCAT scope" states there are 6 main classes. The text and Figure 1 both show 7 classes. Is one class not a main one, or are there 7?

Or am I missing something?

Creator: @chris-little

@agbeltran

This comment has been minimized.

Copy link
Member

@agbeltran agbeltran commented Oct 24, 2019

Thanks for checking this.

The class that is not listed in section 5.1 is dcat:Relationship.

This class is 'at risk' (as indicated in 6.1 https://www.w3.org/TR/vocab-dcat-2/#Class:Relationship) pending evidence of its implementation.

In any case, it would be good to add a clarification, and take your comment into account once a decision is made around dcat:Relationship to have consistency across the document. This will include updating the text with the number of main classes and the figure in 5.1.

@chris-little

This comment has been minimized.

Copy link
Author

@chris-little chris-little commented Oct 24, 2019

@agbeltran Thanks.

@andrea-perego

This comment has been minimized.

Copy link
Contributor

@andrea-perego andrea-perego commented Oct 27, 2019

Actually, dcat:Relationship is not one of the key classes around which DCAT2 is built, as it has been specifically defined as a means to specify qualified relationships between resources. So, I would not consider it as a main class for DCAT2.

@dr-shorthair

This comment has been minimized.

Copy link
Contributor

@dr-shorthair dr-shorthair commented Oct 28, 2019

We can remove it from Figure 1. if you like.

@chris-little

This comment has been minimized.

Copy link
Author

@chris-little chris-little commented Oct 29, 2019

@andrea-perego and @agbeltran have resolved my issue and the action to update is being tracked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.