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

causally_upstream_of _or _within -o- part _of -> causally_upstream_of _or _within #527

Closed
ukemi opened this issue Aug 15, 2023 · 12 comments
Closed

Comments

@ukemi
Copy link

ukemi commented Aug 15, 2023

With the decision to eliminate the 'involved in' terms and convert them to GO-CAM models we are losing information when we convert the GO-CAM to standard annotations. If the original annotation had a GP to term relation of causally_upstream_of_or_within 'blah involved in blac' and it is decomposed to an expression in which the GP is causally upstream of or within blah that is part of blac, only a single annotation is generated to blah. The information about the other process is lost in an annotation extension.

We need to generate annotations to both process as happens when the GP2term relation is part_of.

See http://noctua.geneontology.org/editor/graph/gomodel:64d5781900001000
and the annotation preview for an example.

ping @LiNiMGI

@balhoff
Copy link
Member

balhoff commented Aug 15, 2023

@ukemi are you proposing we add the property chain causally_upstream_of _or _within o part _of -> causally_upstream_of _or _within? I think that makes sense. Could you make an RO issue?

@ukemi
Copy link
Author

ukemi commented Aug 15, 2023

Yep. oborel/obo-relations#746

@ukemi
Copy link
Author

ukemi commented Aug 15, 2023

Do you think we also need
acts_upstream_of_or_within o part_of -> acts_upstream_of_or_within?

@balhoff
Copy link
Member

balhoff commented Aug 15, 2023

Do you think we also need
acts_upstream_of_or_within o part_of -> acts_upstream_of_or_within?

Probably so.

@kltm
Copy link
Member

kltm commented Aug 15, 2023

@ukemi Is this part of a project, or something else?

@ukemi
Copy link
Author

ukemi commented Aug 15, 2023

Something else. Unless we consider annotation review due to obsoletion a project.

@kltm
Copy link
Member

kltm commented Aug 15, 2023

@ukemi Nothing to worry about at your end as it's just metadata, but anything that touches code or the data store (which I believe this is?) should probably be grouped somehow. We have other "projects" for reviews, so something could be made here. I can touch bases w/ @pgaudet when she's back.

@ukemi
Copy link
Author

ukemi commented Aug 15, 2023

@balhoff knows better than I but I suspect once the property chains are added to RO, the reasoning that is in place will consider them and create the missing annotations.

@balhoff
Copy link
Member

balhoff commented Aug 15, 2023

@balhoff knows better than I but I suspect once the property chains are added to RO, the reasoning that is in place will consider them and create the missing annotations.

That's true, it should only require changes in RO.

@pgaudet
Copy link

pgaudet commented Sep 7, 2023

If this only requires changes in RO, should we close this ticket?

@balhoff
Copy link
Member

balhoff commented Nov 13, 2023

I opened a pull request: oborel/obo-relations#771

@balhoff
Copy link
Member

balhoff commented Aug 22, 2024

This has been merged in RO.

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

No branches or pull requests

4 participants