Skip to content

Sort the output of SPARQL checks. - #38

Merged
gouttegd merged 1 commit into
mainfrom
sort-sparql-output
Oct 8, 2026
Merged

gouttegd merged 1 commit into
mainfrom
sort-sparql-output

Conversation

@gouttegd

@gouttegd gouttegd commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

This PR adds a ORDER BY to the SPARQL report queries, so that their output is deterministic. This makes it easier to compare reports from different runs of make all_reports.

(This does not include the -violation.sparql queries; this should not be necessary for those queries since they are supposed to return nothing anyway -- if they do return something, then something is wrong.)

closes INCATools/ontology-development-kit#1385

Adds a `ORDER BY` to the SPARQL report queries, so that their output is
deterministic. This makes it easier to compare reports from different
runs of `make all_reports`.

(This does _not_ include the `-violation.sparql` queries; this should
not be necessary for those queries since they are supposed to return
nothing anyway -- if they do return something, then something is wrong.)

closes INCATools/ontology-development-kit#1385
@gouttegd gouttegd self-assigned this Oct 1, 2026
@gouttegd
gouttegd requested a review from matentzn October 1, 2026 12:14
@gouttegd
gouttegd merged commit 759cec7 into main Oct 8, 2026
6 checks passed
@gouttegd
gouttegd deleted the sort-sparql-output branch October 8, 2026 08:13
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.

Noisy diffs for sparql QC reports

2 participants