CommonCoreOntology / CommonCoreOntologies

The Common Core Ontology Repository holds the current released version of the Common Core Ontology suite.
BSD 3-Clause "New" or "Revised" License
194 stars 58 forks source link

Deconflicting IBEs and ICEs #160

Closed neilotte closed 2 months ago

neilotte commented 1 year ago

Only ICEs can cco:represents, but a few definitions in CCO imply IBEs can represent. I recommend their definitions are adjusted to clarify that this is not the case:

jonathanvajda commented 1 year ago

Aren't the first two properly subclasses of ICE, not IBE? If so, then if someone still wants to talk about the bearers, then just talk about them as subclasses of cco:Document or perhaps defined class with an axiom such as cco:Document bearer_of cco:List

For cco:Chart rather than say 'represents', should it say

?

mark-jensen commented 1 year ago

@neilotte and @jonathanvajda We are currently in the process of reviewing several of the subclasses of ‘Information Bearing Artifact’, considering how to refactor where appropriate. If these classes end up with specific IBA counterparts, the defs will certainly be redone to properly reflect that they carry the thing that represents. Thanks!

jonathanvajda commented 6 months ago

This topic is of renewed interest on both fronts @mark-jensen. A partner of mine just found this same issue that Neil raised. I am also working with ontology working group where this IBE-ICE issue has been under discussion.

neilotte commented 2 months ago

@johnbeve May I ask you to look at this PR thread when you have a moment?

https://github.com/CommonCoreOntology/CommonCoreOntologies/pull/319

johnbeve commented 2 months ago

@neilotte Comments in the #319 PR sir

neilotte commented 2 months ago

This issue was addressed in https://github.com/CommonCoreOntology/CommonCoreOntologies/pull/319 . Closing now.