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

Added the changes in the initial context #745

Closed
wants to merge 1 commit into from

Conversation

iherman
Copy link
Contributor

@iherman iherman commented May 23, 2017

See Official Announcement for the details

@gromgull
Copy link
Member

The current state of the RDFa and mData parser is unfortunately that it has drifted out of sync with the stand-alone versions you maintain (as discussed in #443)

No one has time to sort this mess out. I would vote for removing RDFa and mData from core rdflib for the 5.0 release.

They will still be a simple pip install away, and with the (now many years old!) plugin architecture, people do not have to change their code for this to work.

This also matches the way we deal with json-ld and lots of other extra modules.

@iherman
Copy link
Contributor Author

iherman commented May 29, 2017

@gromgull I understand and see that it is a bit messy but, unfortunately, the separate mData and RDFa repositories are not equivalent. Those are the original programs that are built on top of RDFLib: their usage pattern is that the user runs some dedicated function parsing, say, and HTML file and returning an RDF(Lib) Graph. They are not based on the plugin architecture of RDFLib, ie, they will not be usable from the standard parse function of RDFLib. In this sense, it is not equivalent to the way json-ld is handled.

I agree it would be better to remove that part from the RDFLib code and put both of them in their respective repository, but that is an extra work that I simply do not know if and when I could/would do. In other words, we are a bit stuck:-(

What is the best way of moving forward?

@gromgull
Copy link
Member

We've removed RDFa now - I will open PRs for rdfa and microdata to keep them working as rdflib plugins!

@gromgull gromgull closed this Oct 27, 2018
@gromgull gromgull deleted the rdfa-initial-context-update-2017 branch October 30, 2018 09:21
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 this pull request may close these issues.

2 participants