-
Notifications
You must be signed in to change notification settings - Fork 22
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
Investigations should be specializations of Event #138
Comments
I would object to Investigation being made a subClass of Event and especially any proposal to remove it as a subClass of ContextualCompilation. It is true that one aspect of an Investigation is a set of actions that occur overtime in the investigative process but it is not only that. It is also the set of data/information/knowledge analyzed and the set of data/information/knowledge learned in the investigation. The very heart of an investigation is a contextual compilation of things that happened during the investigation (perdurant things) and things that were observed, collected, analyzed, and learned during the investigation (endurant things). I would not object to making Investigation a subclass of both ContextualCompilation and Event as long as core:eventType was hard set to "Investigation" and the cardinality of core:eventContext was set to maxCount=0 to prevent confusion between it and core:object on ContextualCompilation. |
This patch is exploratory. A follow-on patch will regenerate Make-managed files. References: * casework/CASE#138 * ucoProject/UCO#544 Signed-off-by: Alex Nelson <[email protected]>
References: * casework/CASE#138 * ucoProject/UCO#544 Signed-off-by: Alex Nelson <[email protected]>
Background
UCO Issue 541 introduced general
uco-core:Event
s.CASE's
Investigation
class has, to date, been encoded as auco-core:ContextualCompilation
. (This class design predates my own involvement in CASE, so I can only guess to motives for this.)ContextualCompilation
provides a "Set" construct that lets arbitrary items be grouped together, usinguco-core:object
.Requirements
Requirement 1
Investigation
should exerciseEvent
.Requirement 2
Investigations
must maintain the ability to linkInvestigativeAction
s,ProvenanceRecord
s, and other CASE classes that tie to investigative contexts.This proposal does not impose a requirement they need to be linked with the same predicate as today, but other developments in UCO might mean they would no longer be linked the same way.
Risk / Benefit analysis
Benefits
Investigation
is more a representation of the set of things pertaining to an investigation, rather than the investigation itself.Risks
Event
s could be required to behave differently fromContextualCompilation
s, so this might not be a backwards-compatible proposal. UCO Issue 544 introduces a disjoint pair of classes, endurants and perdurants. Under that proposal,Event
would fall underPerdurant
. While not written in the current proposal's state, it is likely thatCompilation
would be interpreted as anEndurant
. If all that is accepted, anInvestigation
would not be able to be aContextualCompilation
andEvent
, as the classes would be disjoint.participatesIn
, domain endurant, range perdurant; andperdurantProperPartOf
, domain perdurant, range perdurant. This would possibly change howInvestigativeAction
s relate toInvestigation
s, as they may need to useperdurantProperPartOf
(or some subproperty), while anything else (e.g.ProvenanceRecord
s) would need to useparticipatesIn
.uco-core:object
is a vaguely-described linking predicate with no RDFS domain specified and a nearly-maximally generalUcoObject
range, so it is not clear if existing data would need to be changed.Competencies demonstrated
(Competencies deferred for discussion.)
Competency 1
Competency Question 1.1
Result 1.1
Competency Question 1.2
Result 1.2
Solution suggestion
For CASE 1.x.0, BEFORE merging of UCO Issue 544, add to
Investigation
's definition:AFTER merging of Issue 544 (assuming UCO Ontology Committee votes so), subtract this subclassing:
uco-core:object
can retain its usage onInvestigation
, but I suggest this is thanks to a lack of specification, which does not feel future-proofed. This SHACL shape can be added to maintain current data behavior, but Issue 544 could also obviate the property:investigation:Investigation sh:property [ a sh:PropertyShape ; sh:class uco-core:UcoObject ; sh:nodeKind sh:IRI ; sh:path uco-core:object ; ] ; .
Note: That shape also omits the minimum-1 constraint on
uco-core:object
. I am not sure if that constraint was purposefully intended to fail SHACL validation on an investigation containing no objects, or if it was an overzealous translation of anowl:someValuesFrom
restriction.By the time of UCO 2.0.0's release and CASE 2.0.0's release, I think
Investigation
should no longer be aContextualCompilation
.Coordination
develop
for the next releasedevelop
state with backwards-compatible implementation merged intodevelop-2.0.0
develop-2.0.0
(or N/A)The text was updated successfully, but these errors were encountered: