-
-
Notifications
You must be signed in to change notification settings - Fork 66
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
Improve SPARQL tests #248
Comments
The files involved are (some examples to be found here):
We can create a testsuite that use this files to load the data, run the query and compare the results. Full details at RDF-tests. |
Excellent, that's great @Mec-iS ! Also, we can instrument for performance analysis when running these SPARQL test suite features:
For example, these get performed in https://github.com/DerwenAI/ray_tutorial/blob/main/pi.ipynb This will become especially important when we're working with the NVIDIA-basd GPU optimizations for |
Thank you! The SPARQL W3C test suite format seems indeed to be the best. It is implementation independant and a lot of systems implement it like rdflib, Jena, RDF4J, ruby-rdf, sparql-ex and Communica. So, all the efforts done here could be reused to test other systems. About instrumentation, if we instrument from Python we would need to use tools able to instrument native code too. Oxigraph is implemented in Rust and only the rdflib wrapper is in Python. Oxigraph already uses LLVM AddressSantizer to fight memory errors and leaks and Criterion for speed benchmarking. I personally use the Clion profiler to do profiling but something that could be publicly shared would be much better. |
I was thinking writing our own manifest parser but it is better to re-use the Also I will probably copy the |
@Tpt I am running into this error when passing query results into the
It seems that the tests return sometimes return |
@Mec-iS I believe it's because the |
The folders Test runner: if you're willing to use PHP, there is https://github.com/BorderCloud/TFT that powers http://sparqlscore.com/ |
@VladimirAlexiev thanks for your comment Are all the tests in |
after a closer look, there were already I have added more sparql11 tests #259 |
As per conversation.
oxigraph/oxrdflib#8 (comment)
The text was updated successfully, but these errors were encountered: