apache / jena

Apache Jena
https://jena.apache.org/
Apache License 2.0
1.12k stars 652 forks source link

SHACLC: add `name, description, group, order, defaultValue` #2853

Open VladimirAlexiev opened 6 days ago

VladimirAlexiev commented 6 days ago

Version

5.2.0

Feature

Get https://github.com/Sveino/Inst4CIM-KG/tree/develop/source/CGMES/v3.0/SHACL/ttl (these are complex electrical CIM/CGMES shapes). Try it in the example in https://github.com/Sveino/Inst4CIM-KG/issues/137

shacl.bat p -out=c 61968-13_GeographicalLocation-AP-Con-Complex-SHACL_v3-0-0.ttl

You see that the https://www.w3.org/TR/shacl/#nonValidation props name, description, group, order, defaultValue are missing (only message is present).

Are you interested in contributing a solution yourself?

None

afs commented 6 days ago

https://github.com/Sveino/Inst4CIM-KG/issues/137

SHACLC allows Turtle content

Not true.

afs commented 6 days ago

See https://w3c.github.io/shacl/shacl-compact-syntax/#grammar-section

SHACL-C does not include Non-Validating Property Shape Characteristics.

Lets's see what the SHACL 1.2 Working Group produces. It may be the the current SHACL-C is not suited to ful SHACL (inc the decomposed AF).

VladimirAlexiev commented 6 days ago

Agree (see comments in #2852).

But adding these props is easy ("name, description" are "siblings" of "message"), so if jena does it, there's a higher chance others will follow.

afs commented 5 days ago

Discussion on the direction of jena-shacl on #2852.