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

rdflib 6.1.0 and 6.1.1 fail unit tests #34

Closed
hsolbrig opened this issue Jan 15, 2022 · 4 comments · Fixed by #64
Closed

rdflib 6.1.0 and 6.1.1 fail unit tests #34

hsolbrig opened this issue Jan 15, 2022 · 4 comments · Fixed by #64

Comments

@hsolbrig
Copy link
Collaborator

It turns out that the 6.1 release of rdflib makes a mess of the prefix bindings. A graph now comes "out of the box" with 40+ prefixes.

@cmungall
Copy link
Collaborator

This is the rdflib issue: RDFLib/rdflib#1679

@hsolbrig
Copy link
Collaborator Author

hsolbrig commented Apr 9, 2022

This issue has been fixed by the inclusion of rdflib-shim and the somewhat redundant change to the pipenv. OK to close or should we let it sit until the issue above is reconciled one way or the other?

@cmungall
Copy link
Collaborator

cmungall commented May 2, 2022

the rdflib folks have still to make a new release

meanwhile it's impossible to combine sparqlwrapper and funowl:

RDFLib/sparqlwrapper#219

@cmungall
Copy link
Collaborator

cmungall commented Aug 7, 2023

@hsolbrig it looks like you added something to skip tests if the rdflib version is 6.1.1, since this is the version that pollutes namespaces. However, it looks like versions from 6.3 onwards regress to polluting namespaces (see https://groups.google.com/g/rdflib-dev/c/2Dg7MkldV1c), so I will modify this test to skip these versions too.

I will close this issue, but to be safe the next funowl release will be a new minor version rather than patch release

cmungall added a commit that referenced this issue Aug 8, 2023
cmungall added a commit that referenced this issue Aug 8, 2023
cmungall added a commit that referenced this issue Aug 8, 2023
remaining namespace pollution issue, fixes #34. Added tests for 34
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

Successfully merging a pull request may close this issue.

2 participants