-
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
Release 1.3.0 #580
Merged
Merged
Release 1.3.0 #580
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Release 1.2.0
A follow-on patch will regenerate Make-managed files. References: * #536 Signed-off-by: Alex Nelson <[email protected]>
References: * #536 Signed-off-by: Alex Nelson <[email protected]>
Added new core:Event class with appropriate SHACL shapes Added new core:eventType property Added new core:eventContext property Added new core:eventAttribute property
Signed-off-by: Alex Nelson <[email protected]>
Issue 536: Declare and warn of observable:File and :URL disjointedness
References: * #541 Signed-off-by: Alex Nelson <[email protected]>
Issue 541: Add Event
In an OWL-only sense, the ontology portion of this patch could have been accomplished by adding one triple to `action:Action`. For the sake of symmetry and explicitness, `core:Event` also picked up the `owl:disjointWith` statement. To satisfy OWL syntactic requirements, a stub reference to `action:Action` is added, as was done for `types:Dictionary` for Issue 541 (discussed in PR 542). To enforce disjointedness with SHACL, a new independent shape is added, `action:Action-disjointWith-Event-shape`. A follow-on patch will regenerate Make-managed files. EDIT 2023-11-22: An initial version of this patch included an inlined anonymous `sh:NodeShape`. I now believe the scenario that shape mitigates (multi-import of a shapes graph leading to SHACL-scoped syntax errors) is unlikely, and I also saw that the noted rationale around `sh:not` was confused with another predicate (`sh:inversePath`). References: * #563 * #542 Signed-off-by: Alex Nelson <[email protected]>
References: * #563 Signed-off-by: Alex Nelson <[email protected]>
Issue 563: Designate action:Action and core:Event disjoint
A follow-on patch will regenerate Make-managed files. References: * #571 Signed-off-by: Alex Nelson <[email protected]>
References: * #571 Signed-off-by: Alex Nelson <[email protected]>
Signed-off-by: Alex Nelson <[email protected]>
This causes `ValidationResult`s to yield sufficient information to find the whole triple that should have used a sequence. With the prior spelling, only the object of the triple would have been directly identified; the predicate could be manually deduced from the shape name; and the subject was completely absent from the validation result. The noted comment on PR 572 illustrates the difference in results when testing CASE-Corpora. The issue is replicated as new PASS and XFAIL test members. A follow-on patch will regenerate Make-managed files. References: * #571 * #572 (comment) Signed-off-by: Alex Nelson <[email protected]>
References: * #571 * #572 (comment) Signed-off-by: Alex Nelson <[email protected]>
Issue 571: Reduce UCO OWL RDF List review scope to OWL Sequences
Signed-off-by: Alex Nelson <[email protected]>
Update change request template
A follow-on patch will regenerate Make-managed files. Signed-off-by: Alex Nelson <[email protected]>
Signed-off-by: Alex Nelson <[email protected]>
This release PR is ready for review and merge. |
plbt5
approved these changes
Jan 23, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Coordination