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

Add superclass mapping from COB characteristic to ChEBI role #173

Merged
merged 7 commits into from
Sep 14, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/ontology/cob.Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ cob.tsv: cob.owl
# this is a really hacky way to do this, replace with robot report?
.PHONY: sssom
sssom:
pip install sssom
pip install sssom pandasql
pip install --upgrade --no-deps --force-reinstall sssom==0.3.2

$(TMPDIR)/cob-to-external.sssom.owl: $(COMPONENTSDIR)/cob-to-external.tsv | sssom
Expand Down
2 changes: 2 additions & 0 deletions src/ontology/components/cob-to-external.tsv
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,7 @@ COB:0000076 objective specification owl:equivalentClass IAO:0000005 objective sp
COB:0000079 plan specification owl:equivalentClass IAO:0000104 plan specification .
COB:0000080 complex of molecular entities rdfs:subClassOf owl:Thing owl:Thing
COB:0000088 drug product owl:equivalentClass DRON:0000005 drug product we currently use a different text definition but we believe they do not contradict
COB:0000088 drug product rdfs:seeAlso CHEBI:23888 drug These can't be equivalent because COB drug product is a material entity, and ChEBI's drug is a role. See issue #140 for more information.
Copy link
Contributor

Choose a reason for hiding this comment

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

I like this. closeMatch is also not wrong, but I think seeAlso is good as it's even weaker. We should systematize this a bit more

COB:0000101 document owl:equivalentClass IAO:0000310 document .
COB:0000102 data item owl:equivalentClass IAO:0000027 data item .
COB:0000103 cell nucleus owl:equivalentClass GO:0005634 nucleus .
Expand All @@ -103,6 +104,7 @@ COB:0000111 disposition owl:equivalentClass BFO:0000016 disposition .
COB:0000112 function owl:equivalentClass BFO:0000034 function .
COB:0000113 plan owl:equivalentClass OBI:0000260 plan .
COB:0000114 role owl:equivalentClass BFO:0000023 role .
COB:0000502 characteristic sssom:superClassOf CHEBI:50906 role The ChEBI notion of a role doesn't strictly follow the BFO sense of either being a role or disposition, meaning that it's hard, if not impossible, to impose BFO order to this branch of ChEBI. See issue #173
COB:0000116 cellular membrane rdfs:subClassOf owl:Thing owl:Thing
COB:0000118 cellular organism owl:equivalentClass NCBITaxon:131567 cellular organisms .
COB:0000118 cellular organism owl:equivalentClass CARO:0010004 cellular organism .
Expand Down