-
Notifications
You must be signed in to change notification settings - Fork 23
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
Self-referential resource URI for testing manifest documents is incorrect #269
Comments
Other tools have been processing the files successfully for awhile now. Could you explain more please? Taking the TriG syntax tests as an example: if you mean:
There is more than one syntax file for the manifests - Turtle and JSON-LD. They are the same RDF graphs. Relative URIs would mean producing one from the other is not a simple syntax translation. Tools can look for |
Rather than looking up all instances of type
Where the first argument This tool works with a wide variety of tests suites including all standardized RDF 1.1 syntaxes; all of which use relative URIs in the publishing of their JSON-LD and Turtle manifests (e.g. https://w3c.github.io/rdf-tests/trig/manifest.ttl).
Whilst not necessarily being required by the definition, it seems to me that the convention has become that that an instance of I understand the case that having relative URIs would result in more than one graph semantically, but I also don't think it would be semantically problematic to have several identifiers where
My alternative (and perhaps cleanest) reccomendation would be that the webpage https://w3c.github.io/rdf-star/tests/trig/syntax/ should contain (using RDFa) the RDF data that is also contained in https://w3c.github.io/rdf-star/tests/trig/syntax/manifest.ttl, so that when looking up https://w3c.github.io/rdf-star/tests/trig/syntax#manifest the relevant RDF data can be dereferenced. |
@jeswr -- You appear to have a misunderstanding of the meaning of
— indicate that the same entity is identified by the three URIs —
— as well as clearly implying that
We do need to know what the Also, please note that These comments may seem nitpicky and trivial, but for the topics being discussed here, they are absolutely vital to clear expression and comprehension. |
The manifest at IIRC, the The JSON-LD test suite uses For RDF-star, IIRC, we're concerned about the test suite moving, so that the reported manifest becomes disjoint with it's location. I think when the WG is formed, a space in w3.org should be mirrored, and context-negotiation used so that a fetch to (say) |
The link is line 46 of ManifestLoader.ts. At lines 47-53 there is code to deal with RDFa. The same approach could be applied for RDF-star:
Looking for |
I'm finally coming back to this - and to me, there is still one key problem which is that in the root manifest (https://w3c.github.io/rdf-star/tests/manifest.ttl) we have the statement
The tool rubensworks/rdf-test-suite.js#77 expects that the entries in this list are resources of type What this also means is that if I were to load all of the RDF test suites into a single KG I would not be able to run the rdf-star test suites (whilst I expect I could run the others); because this test suite relies on the implicit semantics of the document structure in which it is published in order to define which resources correspond the manifest entries. I appreciate that the WG is only now just kicking off so this may not be possible to address immediately, but it is a discussion that I would like to have at some point :) |
The manifest documents for the test suites are located at URLs like:
https://w3c.github.io/rdf-star/tests/trig/syntax/manifest.ttl
But in the manifest document they refer to themselves to using a fragment identifier e.g.
https://w3c.github.io/rdf-star/tests/trig/syntax#manifest
This results in problems in tools trying to use the manifest automatically, in particular rubensworks/rdf-test-suite.js#77
My suggestion would be to use the relative URI
<>
to refer to the manifest within the document itself.The text was updated successfully, but these errors were encountered: