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

vis.js needs to be reimplemented to use rdflib.js #5

Open
jamsden opened this issue Dec 5, 2018 · 1 comment
Open

vis.js needs to be reimplemented to use rdflib.js #5

jamsden opened this issue Dec 5, 2018 · 1 comment

Comments

@jamsden
Copy link

jamsden commented Dec 5, 2018

vis.js needs to use db.query() to get the triples, and then convert IndexedFormula statements into the graph.

Without doing this, the web page won't successfully load, but you can still test the server with curl or Postman:

curl --request GET \
 --url http://localhost:3000/univ/umaine/students \
 --header 'Accept: text/turtle'
@jamsden
Copy link
Author

jamsden commented May 22, 2019

The old LDPjs implementation of vis.js accessed the raw triples in the database. This is why the underlying database was exposed to the app. However, this is not a desirable solution as it depends directly on the storage service implementation and doesn't use any OSLC client or query abstraction.

The storage service query method implementation is incomplete, so that's not an option. A simple implementation that is similar to what was in LDPjs, and therefore preserves more of the vis.js implementation, would be to use rdflib to read all the triples and then just update the vis.js implementation to get the data from rdflib statements.

But storage services doesn't expose rdflib or any method that returns all triples - which would be a strange method.

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

No branches or pull requests

1 participant