-
Notifications
You must be signed in to change notification settings - Fork 34
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
UCO semi-open vocabularies seem to use owl:onDatatype
incorrectly
#593
Comments
References: * #593 Signed-off-by: Alex Nelson <[email protected]>
PR 628 has been posted implementing the bugfix pertaining to this Issue. A PR adding a shape for OWL syntax review will come in a future PR. The change proposal form is in this comment, for the sake of generating email notices. RequirementsRequirement 1
These syntactic requirements are given in OWL 2 Web Ontology Language Mapping to RDF Graphs (Second Edition). See Table 1 for the constructs' translations to RDF, and note the requirements on blank nodes and accompanying properties.
Non-requirementThis proposal does NOT include a SHACL test for these OWL syntax constructs, but one is planned for a future date. Risk/benefit analysisBenefits
Risks
Solution suggestionDelete |
This proposal also has an impact on the implementation of Issue 549, as that issue adds a new vocabulary. |
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
A follow-on patch will regenerate Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
A follow-on patch will regenerate Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
References: * #549 * #593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
A follow-on patch will regenerate Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
References: * ucoProject/UCO#549 * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
A follow-on patch will regenerate Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#549 * ucoProject/UCO#593 * ucoProject/UCO#612 Signed-off-by: Alex Nelson <[email protected]>
References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
References: * ucoProject/UCO#593 Signed-off-by: Alex Nelson <[email protected]>
A follow-on patch will regenerate Make-managed files. References: * ucoProject/UCO#549 * ucoProject/UCO#593 * ucoProject/UCO#612 Signed-off-by: Alex Nelson <[email protected]>
References: * ucoProject/UCO#549 * ucoProject/UCO#593 * ucoProject/UCO#612 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#549 * ucoProject/UCO#593 * ucoProject/UCO#612 Signed-off-by: Alex Nelson <[email protected]>
No effects were observed on Make-managed files. References: * ucoProject/UCO#549 * ucoProject/UCO#593 * ucoProject/UCO#612 Signed-off-by: Alex Nelson <[email protected]>
Bug description
The document OWL 2 Mapping to RDF Graphs specifies the subtractive parsing process used to parse an RDF graph as an OWL ontology.
When using an externally-developed tool to try reviewing UCO's current OWL conformance, I saw some issues reported around the semi-open vocabularies. See e.g.
vocabulary:BitnessVocab
, chosen for illustration here only because it is short:It appears
owl:onDatatype
is incorrect to use in this definition. From reviewing the OWL2--RDF document noted above, the only OWL construct that predicate appears in isDatatypeRestriction
(Table 12, last row; Table 1, a sole row in the middle).Due to the subtractive nature of the OWL parsing process (described recently in the competencies under Issue 591, Result 1.1), that predicate will be left "dangling" after the structure pertaining to
owl:oneOf
is consumed and removed from the graph. (The reverse seems unlikely, because noowl:withRestrictions
predicate is present.)It appears it is necessary to remove all occurrences of
[] owl:onDatatype xsd:string .
from the UCO (and CASE) vocabulary namespaces.Steps to reproduce
This is among the issues reported from ROBOT 1.9.512, using the following command from UCO's root directory:
The change proposal form is posted in a comment below.
Coordination
develop
for the next UCO releasedevelop
for the next CASE releasedevelop
state with backwards-compatible implementation merged intodevelop-2.0.0
for UCOdevelop
state with backwards-compatible implementation merged intodevelop-2.0.0
for CASEdevelop-2.0.0
(N/A)develop
branch updated to track UCO's updateddevelop
branchdevelop-2.0.0
branch updated to track UCO's updateddevelop-2.0.0
branchFootnotes
Participation by NIST in the creation of the documentation of mentioned software is not intended to imply a recommendation or endorsement by the National Institute of Standards and Technology, nor is it intended to imply that any specific software is necessarily the best available for the purpose. ↩
This Issue is not a suggestion to add that tool to UCO's test suite. ↩
The text was updated successfully, but these errors were encountered: