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

skolemization across loaded graphs #42

Closed
marc-portier opened this issue Apr 8, 2024 · 2 comments
Closed

skolemization across loaded graphs #42

marc-portier opened this issue Apr 8, 2024 · 2 comments
Assignees
Milestone

Comments

@marc-portier
Copy link
Member

this is a revisit of the earlier mentioned #32

in addition to that previous report we delibirately want to test and guarantee this particular case:

-- when two distinct graphs are each of them including blank nodes
-- then those blank nodes need to be distinct

currently we only tested introducing multiple blank nodes inside one same graph

@marc-portier marc-portier added this to the 0.0.2 milestone Apr 8, 2024
@marc-portier marc-portier self-assigned this Apr 8, 2024
@marc-portier marc-portier changed the title skolemnization accross loaded graphs skolemnization across loaded graphs Apr 8, 2024
@marc-portier
Copy link
Member Author

marc-portier commented Apr 9, 2024

looks like we finally found a smoking gun for this problem (after many attempts)

apparently this bnode identifier reuse issue only emerges when inserting graphs that were loaded from json-ld input
this is due to a bug in rdflib.plugins.parsers.jsonld.py

see RDFLib/rdflib#2760

we still have to figure out how we can get around this problem in an elegant way
a dirty (and resource consuming, but solid?) fix seems to be that we can just serialize the graph to turtle, and parse it again.

@marc-portier marc-portier changed the title skolemnization across loaded graphs skolemization across loaded graphs Apr 9, 2024
@marc-portier
Copy link
Member Author

closed in #43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant