barnard59 icon indicating copy to clipboard operation
barnard59 copied to clipboard

pipeline to check class constraints

Open giacomociti opened this issue 2 years ago • 2 comments

an experiment (still a draft) to overcome the issue described in https://github.com/zazuko/barnard59/pull/236.

The new check-class command can check for a class constraint like the one in https://github.com/zazuko/cube-link/blob/main/constraint.ttl

cat cube.ttl | npx b59 cube check-class \
    --path http://ns.bergnet.org/dark-horse#room \
    --class http://schema.org/Place

Of course https://github.com/zazuko/cube-link/blob/main/cube.ttl is small enough and can be checked all at once disabling batching, the command is meant for cases with a huge number of observations.

The main idea is to scan the data filtering only relevant quads and sorting them such that each type declaration is followed by the quads referencing it. The checking step can then easily spot the invalid quads (those missing the corresponding type declaration)

giacomociti avatar Dec 15 '23 14:12 giacomociti

⚠️ No Changeset found

Latest commit: dead75d1fbbc2efd337044e709e1495884a6833c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Dec 15 '23 14:12 changeset-bot[bot]

Codecov Report

Attention: 74 lines in your changes are missing coverage. Please review.

Comparison is base (8ffdff1) 85.54% compared to head (e7fba2d) 84.49%. Report is 41 commits behind head on master.

:exclamation: Current head e7fba2d differs from pull request most recent head dead75d. Consider uploading reports for the commit dead75d to get more accurate results

Files Patch % Lines
packages/cube/lib/classConstraint.js 0.00% 55 Missing :warning:
packages/shacl/report.js 0.00% 19 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #238      +/-   ##
==========================================
- Coverage   85.54%   84.49%   -1.05%     
==========================================
  Files         164      165       +1     
  Lines        5990     6064      +74     
  Branches      100      100              
==========================================
  Hits         5124     5124              
- Misses        864      938      +74     
  Partials        2        2              
Flag Coverage Δ
barnard59 80.29% <ø> (ø)
barnard59-base 37.47% <ø> (ø)
barnard59-core 92.50% <ø> (ø)
barnard59-csvw 80.21% <ø> (ø)
barnard59-cube 89.23% <0.00%> (-7.65%) :arrow_down:
barnard59-formats 32.63% <ø> (ø)
barnard59-ftp 85.38% <ø> (ø)
barnard59-graph-store 94.65% <ø> (ø)
barnard59-http 87.50% <ø> (ø)
barnard59-rdf 94.34% <ø> (ø)
barnard59-s3 96.02% <ø> (ø)
barnard59-shacl 52.86% <0.00%> (-7.28%) :arrow_down:
barnard59-sparql 82.60% <ø> (ø)
barnard59-validation 97.53% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Dec 16 '23 10:12 codecov-commenter