-
Notifications
You must be signed in to change notification settings - Fork 34
toRdf and fromRdf algorithms need WebIDL API methods defined #27
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
Comments
Is this editorial, or does it need discussion? |
Editorial, but we may discuss things in the PR review. |
…or RdfDataset, RdfGraph, RdfTriple, and RdfLiteral. Fixes #27.
Mostly as an aside: the RDF.js CG is bringing in lots of code and working it toward conformance with the Interface Specification: RDF Representation. There's no existing conflict (that I've seen) between this WebIDL and their JS API, but it might serve both communities well if we keep each other appraised of any overlap (if/when it occurs). Thanks for your work on this, @gkellogg! |
@BigBlueHat thanks for the reference, I was aware of this, but had forgotten the details. Technically, we should probably be using interfaces like RDF source, which the RDF Representation describes using the The Interestingly, they don't really have the equivalent of an But, the RDF Representation spec, I think, is more designed to describe actual JavaScript implementations, rather than an abstract model for dealing with RDF concepts. |
Quite likely. The spec itself has also set dormant for quite awhile. I believe folks in the CG are hoping to revisit that work once they've consolidated some of the implementing code projects into the CG's org. But given that it will only ever (by design) be JS-centric, you're probably quite right. Something to keep an eye on (and people to work with), but probably not a reason to change course per se. |
…or RdfDataset, RdfGraph, RdfTriple, and RdfLiteral. Fixes #27.
…or RdfDataset, RdfGraph, RdfTriple, and RdfLiteral. Fixes #27.
No description provided.
The text was updated successfully, but these errors were encountered: