-
Notifications
You must be signed in to change notification settings - Fork 32
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
No roundtrip with property-based data indexing #514
Comments
I had thought I found another form of the same issue, but the issue I was tracking was a flaw in the JSON Schema RDF context, which has since been corrected. |
It's falling afoul of logic in the Term Selection algorithm step 9 where it things that var should be an IRI or IRI reference. Something is needed for this case, and it's odd that it didn't come up before now. I'd say that this is a spec error with an unknown resolution at this time. |
If the problem is roundtripping, it seems to me that there's another related problem in this example: when compacting with the full URI as index property, the result is as follows (on the playground): {
"@context": {
"ex": "http://example.org/ns/",
"prop": {
"@id": "ex:prop",
"@container": "@index",
"@index": "http://example.org/ns/name"
}
},
"prop": {
"@none": [
{
"@id": "ex:bar",
"ex:name": "bar"
},
{
"@id": "ex:foo",
"ex:name": "foo"
}
]
}
} The |
Summary: The value of |
… to first expand container key before compacting it. Fixes #514.
@iherman this may or may not be considered an editorial change. Also, the description doesn't render properly in the errata doc. |
I've updated my implementation and deployed it on the Ruby RDF Distiller. |
… to first expand container key before compacting it. Fixes #514.
…ss-3 changes. (#616) * Add candidate change and insertions for #380/#388. * Reorder change details with newest first. * Comment out 2021-01-05 editorial changes to node-map-generation algorithm. * Comment out 2021-02-25 editorial changes to IRI expansion algorithm. * Add Candidate Correction 2 for issue #514. * CANDIDATE CORRECTION 3 for issue 565, PR 566 to "Fix context processing for reverse terms". * CANDIDATE CORRECTION 4 for `extractAllScripts` option to `toRdf()`.
This issue has been spotted in w3c/wot-thing-description#988. I can't yet decide if it is an implementation bug (which appears in both the playground and the Ruby distiller) or a problem in the API spec.
It is caused during compaction, when the
@index
property of a term definition using a compact IRI. Not only does it not roundtrip, but it fails with an "IRI confused with prefix" error.Minimal example (see in playground) :
Note that expansion works fine. Replacing
"@index": "ex:name"
with"@index": "http://example.org/ns/name"
in the compaction context solves the problem.What is embarassing is that the example from the spec suffers from the same problem: it expands correctly, but fails to compact back, with the same error (see in playground).
The text was updated successfully, but these errors were encountered: