search

ISA-tools / stato

This is the development repository for the STATistics Ontology (STATO). For more information and demonstration on the ontology content, please visit its website:
http://stato-ontology.org/
31 stars 8 forks source link

go:molecular function inconsistent with source ontology #66

Closed jpmccu closed 6 years ago

jpmccu commented 6 years ago

In STATO, it is listed as a bfo:function, but in GO itself, it is a bfo:process. These are disjoint at the occurrent/continuant level. This results in inconsistent reasoning when merging STATO with other OBO ontologies, including GO, HP, CMO, and UBERON.

agbeltran commented 6 years ago

In STATO, GO:molecular_functioncomes from the OBI import. In OBI, GO:molecular function is indeed a subclass of BFO:function (see GO:molecular function as part of OBI in OLS).

Checking GO itself, it does not have a direct relationship with BFO (I checked the OWL file and also GO in OLS).

I do see that in HP, molecular_function is a subclass of process(checking in OLS).

So, it seems that the inconsistent comes from how OBI and HP (and possibly the others you mentioned) deal with mapping GO to BFO, and we should follow up by raising the issue with those ontologies.

proccaserra commented 6 years ago

pointing those issues to the OBO foundry now, @jimmccusker thx for reporting @agbeltran thx for investigating further.

agbeltran commented 6 years ago

@jimmccusker I've pushed the new import module from OBI, whose latest release included the fix of this issue that you reported, than you! This is currently in STATO development version in this repository, and it will be included in the upcoming STATO release.