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

Skip RDF parsing of json-ld files, parse them as JSON directly #102

Closed
SimonVanneste opened this issue Feb 12, 2019 · 1 comment
Closed
Labels
enhancement New feature or request

Comments

@SimonVanneste
Copy link
Contributor

Right now (using LDFetch):

  1. JSON-LD file parsed as JSON
  2. Converted to graph representation (?) using included context
  3. Converted to triples (subject, predicate, object)
  4. Converted to JS objects for our use

1-3 happens inside LDFetch. We should just skip steps 2 and 3

@SimonVanneste SimonVanneste added the enhancement New feature or request label Feb 12, 2019
@pietercolpaert
Copy link
Member

Once comunica/comunica#473 is merged into master of Comunica, we should change our source to Comunica. The power of Planner.js should at some point be that it can automatically read from other sources too.

Closing this in favor of #120

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants