Jesse Wright

Results 261 comments of Jesse Wright

So does this mean an update to the cmake file is required?

@MPvHarmelen I just had a glance at the files changed and there is only a commit adding the test file; did you forget to commit the actual code changes? I'm...

In addition to eye-js another couple of pointers for JS land are: - https://www.npmjs.com/package/@jeswr/pretty-turtle for pretty printing RDF/JS quads to turtle or N3. - https://www.npmjs.com/package/rdf-parse for parsing pretty well any...

> Perhaps https://github.com/eyereasoner/eye/issues/90#issuecomment-1686148928 is good enough to produce nice n3 output @jeswr ? Yup - the pretty-turtle library does pretty n3 as well so it should be good for your...

> What is the overall performance impact of this when caching is not very valuable? Might be useful to run [the parser perf checks](https://github.com/rubensworks/jsonld-streaming-parser.js#performance) again with an without this cache....

> as caching is only done on pointers towards shared memory. My main concern here is that if we have a poor configuration of the `lru-cache` then we are preventing...

Thank you for the sample code. I will leave this issue open, as the main reason I opened this issue is in the interest following the [robustness principle / postels...

+1 on @jonchurch's points > How would you expect to see the req.body with ls+json? I feel personally like this would be a new middleware exported by this package if...

Part of the issue is that the configuration of the engine is converting blank nodes to variables (https://github.com/comunica/comunica/blob/bf5e6a2a0807188bd0bab224da1746b9f48c5d33/packages/actor-rdf-resolve-quad-pattern-sparql-json/lib/ActorRdfResolveQuadPatternSparqlJson.ts#L34-L68) which in this particular use case, and IMO most use cases for...

Thanks for the clarifications! The patch allowed me to successfully run the following query with the skolem IRI (using the default setup for the comunica engine with nodejs) ``` SELECT...