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

Move Linked Data Platform into Appendix #873

Merged
merged 9 commits into from Apr 18, 2019
Merged

Conversation

davebrowning
Copy link
Contributor

As per discussion in #254 at #254 (comment)

@davebrowning davebrowning added dcat alignment critical defects that must be completed for CR labels Apr 2, 2019
@davebrowning davebrowning added this to the DCAT CR milestone Apr 2, 2019
@davebrowning davebrowning self-assigned this Apr 2, 2019
@davebrowning davebrowning marked this pull request as ready for review April 2, 2019 12:24
@davebrowning
Copy link
Contributor Author

davebrowning commented Apr 2, 2019

Changes as discussed in #254 - checking my changes to the examples is both good and necessary. Obviously dcat-ld: doesn't exist... is there a better way to express this?

@davebrowning
Copy link
Contributor Author

davebrowning commented Apr 2, 2019

@jakubklimek - in the (time limited ) push to get to a proposed recommendation we've had to slim down a lot of the potential alignment work and ensure we're very clear in what we're including in the immediate release. This includes the LDP relationship work, so I've drafted some mods.. Comments very welcome if you can make the time.

@davebrowning
Copy link
Contributor Author

Rendering available here

Copy link
Contributor

@andrea-perego andrea-perego left a comment

Choose a reason for hiding this comment

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

@davebrowning , after having reviewed again this section, I'm even more convinced that it needs to be consolidated and further reviewed by the WG to be safely included in the CR.

My suggestion is to move it to the backlog, and address it as one of the future DCAT priorities.

@riccardoAlbertoni
Copy link
Contributor

davebrowning wrote:

Changes as discussed in #254 - checking my changes to the examples is both good and necessary. Obviously dcat-ld: doesn't exist... is there a better way to express this?

I think it might be better to use the namespace "dcat-ldp" instead of "dcat-ld", "dcat-ldp" makes probably clearer that is the namespace for linked data platform stuff and not other kinds of linked data publication.

About the inclusion in the CR, I am split, I think it would be great to have something helping in the direction of SOLID, but I have to admit my limited confidence with LDP. So it is ok to have it in the future priorities as suggested by @andrea-perego.

@davebrowning
Copy link
Contributor Author

I've brought this up to date and incorporated the simple suggestions. Suggest we discuss at the next quorate sub-group meeting (or plenary - whichever gets around to it first).

@dr-shorthair
Copy link
Contributor

The proposed additional predicates are quite simple and obvious I think so I'm not too worried about breaking anything. But should be marked non-normative.
Namespace proliferation is undesirable, but may be unavoidable.

Copy link
Contributor

@dr-shorthair dr-shorthair left a comment

Choose a reason for hiding this comment

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

Move to informative annex is an improvement over the current situation so should be done immediately.
We can then decide whether it stays at all.

@davebrowning davebrowning merged commit 6bd54d8 into gh-pages Apr 18, 2019
@dr-shorthair dr-shorthair deleted the dcat-issue254-daveb branch April 22, 2019 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alignment critical defects that must be completed for CR dcat
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants