geneontology / pathways2GO

Code for converting between BioPAX pathways and Gene Ontology Causal Activity Models (GO-CAM)
8 stars 0 forks source link

Reactome: R-HSA-5683569 has multiple issues #146

Closed nataled closed 1 year ago

nataled commented 2 years ago

R-HSA-5683569 (CysO263-ATG3) has a modification at position 263 indicated by MOD:01777. The description of the reaction states "The activated LC3 protein is transferred to Cys-263 of ATG3 through a thioester bond." A number of discrepancies thus arise:

1) Position 263 in ATG3 is a proline. 2) MOD:01777 is specifically for a link between a C-terminal glycine residue to a free cysteine. Each of the 'LC3' proteins (all members of the set, plus the candidate member) has a C-terminal glycine (verified), so that part is okay, but--assuming the correct position is found--the connection would not be to a free cysteine. 3) As the described connection is between two proteins (or, as in this case, between a protein and a member of a set), there should be an indication of that, much like what is done for SUMO etc.

nataled commented 2 years ago

It is likely similar issues exist with R-HSA-5683584.

deustp01 commented 2 years ago

The incorrect residue is straightforward to fix but the rest is complicated (there's a set of proteins buried at the place in the data structure where a modifying group belongs) and will need expert re-curation to untangle. We do have a class that allows us to annotate the covalent joining of two different gene products to form a single physical entity, mostly applied so far to annotate disulfide bonds, but sometimes to annotate other kinds of covalent linkages between side chains of different proteins, e.g., R-HSA-5653752.

This may be an issue for the future. So far for ubiquitination and SUMOylation it's probably OK to ignore the fact that ubiquitin and SUMO are themselves gene products and just treat them as chemical adducts on the target protein, without distorting biology for our users. For these other cross-linking cases, that may not be true. But before we reach that issue, I need to figure out exactly what is getting attached to what in the instances you have flagged.

deustp01 commented 1 year ago

All fixed

These corrections will become visible on the public Reactome site and in the BioPAX download with our next release in December (version 83).

To do, but this is really a separate issue beyond the scope of this ticket: review this material for conformity to current Reactome standard practices for annotating ubiquitination-like processes.

So closing this ticket.