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

Incorrect ID for 'planned process' in cob-to-external #191

Closed
beckyjackson opened this issue Jan 15, 2022 · 5 comments · Fixed by #285
Closed

Incorrect ID for 'planned process' in cob-to-external #191

beckyjackson opened this issue Jan 15, 2022 · 5 comments · Fixed by #285

Comments

@beckyjackson
Copy link
Collaborator

In cob-to-external.tsv, 'planned process' has the ID COB:0000035:

COB:0000035 planned process owl:equivalentClass OBI:0000011 planned process . HumanCurated

... but in the COB file, this is COB:0000082:

COB/cob.owl

Lines 726 to 744 in a2f7141

<!-- http://purl.obolibrary.org/obo/COB_0000082 -->
<owl:Class rdf:about="http://purl.obolibrary.org/obo/COB_0000082">
<rdfs:subClassOf rdf:resource="http://purl.obolibrary.org/obo/BFO_0000015"/>
<rdfs:subClassOf>
<owl:Restriction>
<owl:onProperty rdf:resource="http://purl.obolibrary.org/obo/COB_0000081"/>
<owl:someValuesFrom rdf:resource="http://purl.obolibrary.org/obo/OBI_0000260"/>
</owl:Restriction>
</rdfs:subClassOf>
<rdfs:subClassOf>
<owl:Restriction>
<owl:onProperty rdf:resource="http://purl.obolibrary.org/obo/COB_0000087"/>
<owl:someValuesFrom rdf:resource="http://purl.obolibrary.org/obo/SEPIO_0000048"/>
</owl:Restriction>
</rdfs:subClassOf>
<obo:IAO_0000115>A process that is initiated by an agent who intends to carry out a plan to achieve an objective through one or more actions as described in a plan specification.</obo:IAO_0000115>
<rdfs:label>planned process</rdfs:label>
</owl:Class>

In cob-edit.owl, COB:0000035 is 'completely executed planned process' so can we either update the ID or label in the external mapping file? I don't know if this mapping was intended to be to the completely-executed one or not. @cmungall ?

@bpeters42
Copy link
Contributor

I would bet that this was an accident. But it reminds me that we have to be careful when doing the mapping to OBI, where what we have as 'planned process' is really 'completely executed'. Which further reminds me that we should do an OBI-COB alignment.

@bpeters42
Copy link
Contributor

@sebastianduesing - can you check if this is still a thing?

@cmungall
Copy link
Contributor

I made the PR but I kept this as exact, I think the misalignment deserves it's own issue

@sebastianduesing
Copy link
Collaborator

I might be misunderstanding this issue, but I'm not sure that #282 is the right fix. My reading of this issue is:

  • In cob-edit.owl, COB:0000035 is 'completely executed planned process'.
  • In cob-edit.owl, COB:0000082 is 'planned process'.
  • OBI:0000011, which is called 'planned process' in OBI, is actually equivalent to COB:0000035, 'completely executed planned process'.
  • The problem with cob-to-external.tsv is that the mapping for OBI:0000011 says it is mapped to a term with the IRI COB:0000035 ('completely executed planned process') but the COB term label in that row is simply 'planned process.'

The change in #282 changes the IRI to align with the label in cob-to-external.tsv. I believe that, because we do want OBI:'planned process' to map to COB:'completely executed planned process', the right fix would actually be to change the label to align with the IRI.

@cmungall
Copy link
Contributor

cmungall commented Sep 19, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants