search

G-Node / python-odml

odML libraries
http://g-node.github.io/python-odml/
BSD 4-Clause "Original" or "Old" License
24 stars 30 forks source link

Move rdflib dependent code to own library #417

Open mpsonntag opened 2 years ago

mpsonntag commented 2 years ago

The rdflib library is required for conversion of odml to its RDF specific format. Since this library has caused multiple issues in the past and is still causing issues to this date it might be worth looking into moving RDF dependent code from the odml core library into a dedicated sub-library.

penguinpee commented 1 year ago

Hi. May I ask what the plans are with regards to rdflib? Version 6.2.0 has landed in Fedora rawhide recently, breaking the build of odml. I noticed odml has the rdflib version pinned to 5.0.0. That would require a compat rdflib5 package in Fedora to satisfy odml's dependencies.

Before we embark on that endeavor I wanted to check with upstream (you) on plans for making odml work with rdflib >= 6.0.0.

mpsonntag commented 1 year ago

Hi @penguinpee, thank you for the comment! This issue has been on my todo list for quite a while now, but unfortunately I cannot say, when I will get around to addressing it as of yet. My plan would be to first fix the build with the latest RDFlib version available. This will probably require a small change in the odml RDF definition regarding list items of which the handling I think has changed in one of the newer RDFlib version. Once that is done I would like to keep RDF in the odml library, but make it an optional installable. Maybe these infos already help a bit in making a decision.

penguinpee commented 1 year ago

My plan would be to first fix the build with the latest RDFlib version available. This will probably require a small change in the odml RDF definition

Excellent. This would also be the easiest way forward for us. Small change sounds good. Would you have an estimate when this small change can be expected? Not trying to push, just gathering info for relaying back to the group.

Thanks for the quick response.

mpsonntag commented 1 year ago

Would you have an estimate when this small change can be expected?

I would like to fix it within the next 2 month, but I am a bit swamped at the moment so please don't hold it against me if it takes a smidge longer.

penguinpee commented 1 year ago

A gentle nudge: not pushing, just wondering if there is any news, any update.

mpsonntag commented 1 year ago

@penguinpee: I think I managed to upgrade to the usage of rdflib >= 6.0.0, but still need to clear some changes with the other core devs and run some tests in the ecosystem before putting up a new PyPI version.

penguinpee commented 1 year ago

Thanks a lot. I'll wait for the updated PyPI package, which should trigger some notification by release monitoring on our end.

mpsonntag commented 1 year ago

A new release (1.5.3) has been uploaded to PyPI that supports rdflib v6+ by default. There should be no changes in behavior. Users that require Python <=3.6 still have to rely on rdflib v5.0.0.