search

RDFLib / pyrdfa3

RDFa 1.1 distiller/parser library: can extract RDFa 1.1 (and RDFa 1.0, if properly set via a @version attribute) from (X)HTML, SVG, or XML in general. The module can be used to produce serialized versions of the extracted graph, or simply an RDFLib Graph.
http://www.w3.org/2012/pyRdfa/
Other
67 stars 22 forks source link

Python3 version #34

Closed iherman closed 4 years ago

iherman commented 4 years ago

Finalized a Python 3 version

iherman commented 4 years ago

@nicholascar thanks. I realized that, actually, there was a discrepancy in the documentation, so I had to regenerate it...

That being said:

I'm not sure this module now works with RDFlib 5.0.0 and for the second, RDFlib 5.0.0 works with Python 2.7 & 3.4+.

I am not sure how the dates got tangled, but the changes were actually made in April. I do not remember exactly when 5.0.0 came out, but that is certainly the version I run on my machine and I tested with. I did not test this with earlier versions of RDFLib, hence that dependency.

As for the 3.4 vs. 3.8: I must admit I do not remember the details. What I do know (because I was pretty upset about it) is that there was some specific, and backward-incompatible change between 3.8 and a previous version, which created some problems with my code. I know I had to fight with this sometimes at the beginning of the year; however, subsequent private events (some health issues, COVID lockdown in France, etc...) wiped out my memory. But I do know that I needed to upgrade my local Python3 version to get things running. So yes, I am afraid Python3.8 is necessary for this module:-(

(Sorry to be a bit vague...)

FlorianLudwig commented 4 years ago

@iherman @nicholascar I tried the lib with python3.5 and python3.6 by running scripts/localRDFa.py on a small html file and had no issues.

With even python 3.5 having reached end of life, I would not spend any time on python 3.4.

nicholascar commented 4 years ago

I’ll merge this shortly and review state of the module in general.

We are preparing to incorporate JSON-LD and a few other bits and pieces into RDFlib core, so if dependencies can be aligned, this can go in too and thus both increase RDFLib capabilities and reduce maintenance burden. We will have to test this integration all properly of course, but that’s do-able.

iherman commented 4 years ago

B.t.w., @nicholascar, I think I have taken care of your issues above, maybe you want to update your review...