obophenotype / upheno

The Unified Phenotype Ontology (uPheno) integrates multiple phenotype ontologies into a unified cross-species phenotype ontology.
https://obophenotype.github.io/upheno/
Creative Commons Zero v1.0 Universal
76 stars 17 forks source link

DP for hypoglycemic coma and similar classes #307

Open cmungall opened 5 years ago

cmungall commented 5 years ago

Carried from https://github.com/ncats/translator-workflows/issues/29

Do we have a DP for phenotype A that arises from phenotype B?

cc @cbizon

drseb commented 5 years ago

Not everyone has access to the linked repo. Can you add more context here?

cmungall commented 5 years ago

check https://github.com/ncats/translator-workflows/invitations for an invite

But there's not that much additional context for you, basically we just need an owl def for this HPO class

drseb commented 5 years ago

I think what you rather need, is an annotation of MONDO:0004946 with HP:0001325 , right? If you agree, what would be the best way? What is the corresponding entry in the annotation-repo?

cmungall commented 5 years ago

I wasn't thinking of MONDO (the confusing fact we have hypoglcyemia in both ontologies can be ignored)

I was thinking of using http://purl.obolibrary.org/obo/HP_0001943 (hypoglycemia) with coma (HP:0001325)

drseb commented 5 years ago

Using for what? Sorry, I am a bit confused.

drseb commented 5 years ago

Are you suggesting that Hypoglycemic coma should be a subclass of Hypoglycemia?

cmungall commented 5 years ago

Nope, it would be a causal relation

eg

Hypoglycemic coma (http://purl.obolibrary.org/obo/HP_0001325) = coma (http://purl.obolibrary.org/obo/HP_0001259) and caused-by some hypoglycemia (http://purl.obolibrary.org/obo/HP_0001943)

(Of course, sometimes phenotype ontologies conflate these causal relations into an overloaded is_a, I'm not suggesting we do it here, but it happens, see #291)

pnrobinson commented 5 years ago

1+ for this causal relation