Skip to content
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

Added "device" #232

Merged
merged 5 commits into from
May 1, 2023
Merged

Added "device" #232

merged 5 commits into from
May 1, 2023

Conversation

ddooley
Copy link
Contributor

@ddooley ddooley commented Mar 24, 2023

To address #222 . Also added "has function" and "function of", understanding that ultimately they will be imported from RO.

Note domain of "has function" is an 'independent continuant' but COB doesn't have that so are we needing a disjunction of top-level terms to reflect comparable logic?

Also added "has function" and "function of"
@ddooley ddooley changed the title Added "device" for #222 Added "device" and "has function" Mar 24, 2023
@ddooley
Copy link
Contributor Author

ddooley commented Mar 24, 2023

I don't know why the cob-edit.owl diff is different. I'm using latest protege 5.6.1. Its coding <owl:annotatedTarget rdf:nodeID="genid58"/> outside of classes they are mentioned in. Maybe that's normal now?!

@cmungall
Copy link
Contributor

We need to have this in the CONTRIBUTING.md but PRs like this should always have corresponding edits to cob-to-external

Copy link
Contributor

@cmungall cmungall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can be merged when spurious diffs resolves (note cob has not been 5.6.1 normalized yet) and when cob-to-ext changes in

@matentzn
Copy link
Contributor

matentzn commented Apr 28, 2023

@matentzn matentzn requested a review from cmungall April 28, 2023 12:59
@ddooley ddooley changed the title Added "device" and "has function" Added "device" Apr 28, 2023
@ddooley
Copy link
Contributor Author

ddooley commented Apr 28, 2023

I've renamed. Forgot that there was shift to using "has characteristic" rather than "has function".

@cmungall
Copy link
Contributor

cmungall commented May 1, 2023

Go ahead and merge if this is a blocker but I think we need a new SOP for NTRs that replace OBO IDs

proposed here:

@ddooley
Copy link
Contributor Author

ddooley commented May 1, 2023

Note that OBI's "device" term is currently specialized to scientific investigations: "A material entity that is designed to perform a function in a scientific investigation, but is not a reagent." so its not an equivalent to the COB device term. I recall OBI will reuse COB's term, and add subclasses for functional kinds of device.

@matentzn matentzn merged commit 72f24d9 into OBOFoundry:master May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants