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

Issue #1308 #1340

Merged
merged 5 commits into from
Apr 6, 2021
Merged

Issue #1308 #1340

merged 5 commits into from
Apr 6, 2021

Conversation

andrea-perego
Copy link
Contributor

@andrea-perego andrea-perego commented Mar 27, 2021

Relevant issue: #1308

Summary of changes:

  • Add dcat:first, dcat:last, dcat:next, dcat:prev to dcat:Resource
  • Update list of inherited properties for subclasses of dcat:Resource
  • Add xhv to the table with informative namespaces
  • Add NOTE after the list of inherited properties of dcat:DatasetSeries, to say that some of the inherited properties may have particular semantics when used in this context
  • Editorial fixes

Preview: https://raw.githack.com/w3c/dxwg/dcat-issue-1308/dcat/index.html

Diff: https://services.w3.org/htmldiff?doc1=https%3A%2F%2Fw3c.github.io%2Fdxwg%2Fdcat%2F&doc2=https%3A%2F%2Fraw.githack.com%2Fw3c%2Fdxwg%2Fdcat-issue-1308%2Fdcat%2Findex.html

@rob-metalinkage
Copy link
Contributor

Do you really want to pull in terms into DCAT as a W3C Rec from a W3C Note that itself declares it is a profile of DCAT?

Understand you need to use these, and not want to re-invent the wheel (and my own implementations do exactly the same) - but the logically consistent approach would be to define a profile of DCAT that uses the parts of ADMS you need, as a profile of ADMS itself probably, that itself has the same status (Note) of ADMS. You then guide people to this rather than add it to the normative Rec status DCAT specification. It appears that a decision about not publishing profiles of DCAT is becoming an arbitrary limitation that leads to anti-patterns where profiling would provide a better option.

@andrea-perego andrea-perego linked an issue Mar 29, 2021 that may be closed by this pull request
@andrea-perego
Copy link
Contributor Author

@rob-metalinkage , I suggest you check the status of the related discussion in issue #1308

@rob-metalinkage
Copy link
Contributor

@andrea-perego ok - i have copied comments in there, as that issue has clearly not addressed this points yet.

@andrea-perego andrea-perego marked this pull request as ready for review March 30, 2021 21:47
@andrea-perego andrea-perego changed the title Add adms:last, adms:next, adms:prev to dcat:Resource Issue #1308 Mar 30, 2021
Copy link
Contributor

@davebrowning davebrowning left a comment

Choose a reason for hiding this comment

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

As discussed, there will likely need to be some further discussion but the updates in the PR seem fine to merge now

@davebrowning davebrowning merged commit 71d961f into gh-pages Apr 6, 2021
@andrea-perego andrea-perego deleted the dcat-issue-1308 branch April 6, 2021 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Any other properties to mention in class dcat:DatasetSeries?
4 participants