-
Notifications
You must be signed in to change notification settings - Fork 8
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
Comments
@ukemi are you proposing we add the property chain |
Do you think we also need |
Probably so. |
@ukemi Is this part of a project, or something else? |
Something else. Unless we consider annotation review due to obsoletion a project. |
@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. |
If this only requires changes in RO, should we close this ticket? |
I opened a pull request: oborel/obo-relations#771 |
This has been merged in RO. |
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
The text was updated successfully, but these errors were encountered: