-
Notifications
You must be signed in to change notification settings - Fork 3
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
GO Centric View #271
Comments
Turns out it is literally hard coded if r['relation'] == 'has_function': data['go']['Molecular Function'][(r['ontology_id'], modifier)].append(r) elif r['relation'] == 'participates_in': data['go']['Biological Process'][(r['ontology_id'], modifier)].append(r) elif r['relation'] == 'located_in': data['go']['Cellular Location'][(r['ontology_id'], modifier)].append(r) Problem is that we seem to be using 'capable_of' now instead of has_function. I can fix by changing the code here.
|
Here is how they seem to be used has_part -> Pfam So the only one that is using GO terms is part_of |
In addition to the relations used in PAF, we also have some used in the OBO. I'll list them here for completeness, then organize them based on function, process, or component considerations in a later comment. Before I do so, however, one question I have is whether or not the annotations shown in the table must come exclusively from the PAF, or does the code allow for these to come from the OBO? If the former, then I might have to move some to the PAF.
|
It currently only shows the PAF |
Okay, we can keep it like that and move any relevant annotations that are currently only in OBO to also appear in PAF. I guess the question then becomes: What do we do with annotations that don't use GO (such as the disease-related annotations)? Right now those don't appear at all. Let's discuss that on the PRO call this Friday. |
My first pass categorization. Note:
Again, to be discussed.
|
The GO-centric view does not show GO Molecular Function terms (Biological Process and Cellular Component terms are OK). For example, see https://proconsortium.org/app/entry/PR:000025934/
The text was updated successfully, but these errors were encountered: