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

RF simple2 query for more exhaustively defined sorting, tuned up formatting of .sparql, new produced output #18

Closed
wants to merge 5 commits into from

Conversation

yarikoptic
Copy link
Member

I have noted in

…twareLabel ?laterality ?softwareLabel ?volume

to achieve clear and deterministic order which would group results per subject nicely and deterministically
…rsion of query

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "bash -c 'GRAPHDB_URL=http://localhost:8889/bigdata; export GRAPHDB_API_URL=\"$GRAPHDB_URL/sparql\"; output=queries/simple2_query_output.csv; /usr/bin/time curl -X POST \"${{GRAPHDB_API_URL}}\" --data-binary \"@queries/simple2_query.sparql\" -H \"Accept: text/csv\" -H \"Content-Type: application/sparql-query\" >| \"$output\"'",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
@yarikoptic yarikoptic requested review from dbkeator and satra February 26, 2024 15:39
@yarikoptic
Copy link
Member Author

now that we moved and I acquired super-powers I will try to push this effort forward by accepting my own PRs if left without statement of showstoppers from requested reviewers. For this one - I would appreciate reviews by this Fri March 1st. Invited @satra and @dbkeator but I would welcome all @ReproNim/trd1 folks and @dnkennedy as well ;-)

@satra
Copy link
Contributor

satra commented Feb 26, 2024

@yarikoptic - the query changes look cosmetic, so fine with me as long as it runs and produces proper output. is the csv based on data in reprolake? or some local store?

@yarikoptic
Copy link
Member Author

Thank you @satra! I think it produces the same (besides order is different as intended).
As for different output values altogether -- it is based on local population as done with what is used/modified in

Is there an endpoint I could expect to run the query against and get old result???

@dbkeator
Copy link
Collaborator

dbkeator commented Feb 26, 2024

Hi,
The CSV output file might be quite old. I never cared about updating it. The wBrainVols branch should contain the same NIDM files as the current version of ReproLake.

The Readme.md file here shows how to upload the NIDM files into a local Blazegraph instance and do the queries. We haven't written up instructions on querying ReproLake but I think @jgrethe could probably give some instruction. I'm not entirely sure how all this will scale as we change NIDM files here and/or change what's in ReproLake, the query returns may get out of sync. This repo was originally just a scratch space for testing, development, etc. The ReproLake was our definitive externally-facing NIDM store....or at least that's how I always thought about it....

?measure a/rdfs:subClassOf nidm:DataElement;
rdfs:label ?softwareLabel;
nidm:measureOf ilxb:ilx_0112559 ;
nidm:datumType ilxb:ilx_0738276 .
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

more indentation would help with readability here

@djarecka
Copy link
Member

djarecka commented Mar 14, 2024

@yarikoptic - I'm testing oxigraph again on the query that Jeff gave and now on this, but I don't have any reasonable times (killing after running for hours without any results), but I'm looking at your script to do it this more automatically

@surchs
Copy link

surchs commented May 22, 2024

Hey folks, I just found this site with some benchmarks of RDF stores: https://github.com/ad-freiburg/qlever/wiki/QLever-performance-evaluation-and-comparison-to-other-SPARQL-engines looks very cool - and somewhat tracks our experiences. That said: the doc is made by the maintainers of one of the tools listed (QLever)

@yarikoptic
Copy link
Member Author

Let's consider this superseded by

@yarikoptic yarikoptic closed this Sep 12, 2024
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

Successfully merging this pull request may close these issues.

5 participants