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
Optional keyword does not seem to work #1316
Comments
Someone will answer your question soon. In the meantime, you might be able to get help more quickly on our Gitter channel. |
I see you're using Comunica v3. Could you try it out on the latest v2 version (2.10.2) to check if the problem already existed back then? (might be a regression in v3) |
I used a 2.10 version previously, which had the same symptoms. |
I see. Very surprising that it does work with link traversal though. Could you check what happens if you move the data to the default graph and remove the GRAPH keyword from the query? Perhaps that's related as well. |
I added a version without graph-embedding in my minimal test case: https://github.com/pietervdvn/sparql-test This one works and gives results as expected. However, I'm reusing data from an upstream provider where I cannot really change the shape of the data they provide... |
Hmm, that's interesting. |
I'm not really sure how to convert this efficiently to TTL. This website ignored everything within the graph. However, I did a simple test query ( |
Issue type:
Question:
I have two json-lds. They both graphs, each containing a geocoordinate (with
schema.org/geo
). Optionally, they might also contain a 'shape' property defining aPolygon
.I'm trying to extract the shape if there is one with a comunica sparql query, which is more or less:
However, this result does not give any bindings if there is no geoshape present - even though this is wrapped into an optional.
When using the
link-traversal
-build withlenient: true
as option, this suddenly works.You can reproduce this issue; I've made a (more or less) minimal example here:
https://github.com/pietervdvn/sparql-test
The text was updated successfully, but these errors were encountered: