-
Notifications
You must be signed in to change notification settings - Fork 47
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
Address 1536: extend the range of 'spatial resolution' #1543
Conversation
the range now allows in addition to . I also added a note to explain the rationale of that change. Should the note be copied in 6.8.13 as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this will require a matching change to the TTL representation
https://github.com/w3c/dxwg/blob/gh-pages/dcat/rdf/dcat3.ttl#L1554
Do I understand it correctly, that the idea is that JSON-LD contexts of DCAT and DCAT profiles can choose one of:
|
Right, but I'm not sure exactly what the process is for this (are some versions generated from others? difference between dcat3.* and dcat3-external.*...) so I would rather leave this to the editors. |
or they can refrain to chose, and leave it to end users... Actually, both options are possible, as demonstrated in this example |
Well, technically, yes. But then the role of JSON-LD in "hiding RDF complexity for JSON users" is somehow diminished if we ask "JSON developers" to choose between "spatialResolutionInMeters": {
"@value": "3.14",
"@type": "xsd:decimal"
} and "spatialResolutionInMeters": {
"@value": 3.14,
"@type": "xsd:double"
} for, in JSON world, no apparent reason. |
@jakubklimek
and that will be compliant with DCAT, thanks to the proposed change. Developers who care about those details still have the opportunities to be more precise and use an |
But this is not that simple. If they go this way, the will probably also use: "spatialResolutionInMeters": 3 which produces a |
@jakubklimek strictly speaking Now, SHACL and ShEX are only concerned with syntax, and a shape requiring I have added a comment (bcb5990) explaining this in the note below the definition of NB: this issue with |
@andrea-perego can you clarify your position on this? |
It is arguably out of scope for DCAT to address issues in the JSON implementation of RDF. |
Co-authored-by: Pierre-Antoine Champin <[email protected]>
Rephasing and removing the normative effect of proposal PR #1543
I have approved this PR as a consequence of the "rephasing and removing of normative effect" PR #1573 just merged here. @dr-shorthair @andrea-perego @agbeltran @davebrowning @pwin : Can any of you approve and merge the resulting PR #1543 into the main branch. |
The range now allows
xsd:double
in addition toxsd:decimal
.I also added a note in 6.6.5 to explain the rationale of that change.
Should the note be copied in 6.8.13 as well?
Addresses #1536
Preview:
Diff: