-
Notifications
You must be signed in to change notification settings - Fork 108
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
Issue with Store.executeWithEnvironment, docs outdated? #119
Comments
Quick update: I derived the named graphs dataset from Store.registeredGraphs and that list contained duplicates. QueryPlanDPSize.executeBGPDatasets fails silently if https://github.com/antoniogarrote/rdfstore-js/blob/master/src/query_plan.js#L460 evaluates to false. |
I am trying to run SPARQL queries on an array of NamedGraphs as shown in the README document. I am still experiencing the issue as mentioned in this issue. I am getting error in at this line: https://github.com/antoniogarrote/rdfstore-js/blob/master/src/utils.js#L90 I will appreciate any idea on this issue. For reference here is the details of the example I am trying: var defaultGraph = [] From registeredGraphs, I am getting these two named graph: [ 'http://example.org/alice', The error I am getting as follows: /Users/smruti/gitrepo/rdfNidm/node_modules/rdfstore/src/utils.js:90 TypeError: uri.indexOf is not a function |
I am trying to run SPARQL queries on the union of all named graphs, providing a defaultDataset like the README shows. This doesn't seem to work. If I construct the dataset with objects
It ends up being wrapped in another object of similar shape and ultimately fails at https://github.com/antoniogarrote/rdfstore-js/blob/master/src/utils.js#L90. Defining a dataset with plain strings in an array does not throw an exception but also yields an empty result. Am I doing something wrong or is this a known issue?
The text was updated successfully, but these errors were encountered: