geneontology / pathways2GO

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

Activity node for reaction should be a process if GO BP is curated at Reactome #318

Open dustine32 opened 3 months ago

dustine32 commented 3 months ago

Here is how the reaction R-HSA-163743 "Transcriptional activation of ACACA by ChREBP:MLX" (the middle box) is currently converted: image

There is no GO MF term or controller so the activity is Molecular Event. Reactome has curated a GO BP term so it is added and connected to the reaction via a "part of" relation.

The new conversion requirements should instead result in this pattern: image The Molecular Event activity is gone with the Reactome-curated GO BP term now anchoring the "has input" and "has output" edges and connected to the preceding reaction via a "part of" relation.

In other words:

  1. Remove edges from the reaction to its preceding reaction (usually a causal relation)
  2. Replace this edge with "part of"
  3. Replace reaction term with curated GO BP term
  4. Don't create a new individual node for the GO BP

image This rule should apply when a reaction class is Molecular Event (no GO MF term) and has a curated GO BP term. As discussed on the 2024-06-13 Reactome weeds call (@ukemi, @deustp01, @vanaukenk can confirm), a curated BP term should also override any inference of transporter activity in the conversion.

The code should decide which relation to connect between the reaction's curated BP node and its preceding activity node by fetching the preceding node's MF term and checking if the BP term is in the MF term's "part of" closure (ancestor terms). If yes, the relation will be "part of," otherwise the relation will be "causally upstream of".

dustine32 commented 3 months ago

@ukemi @deustp01 Plugging in this change and running the tests, our transport test example reaction (R-HSA-201669 in model R-HSA-201681) has changed because it's curated to a GO BP term. Showing the reaction before my change: image And after: image As you can see, the location info has been discarded along with the inferred "transport activity" function term. Is this ok with you?

ukemi commented 3 months ago

Hi @dustine32. Yes and no. This is something we should discuss on a weeds call. For nuclear import, we have been inferring a transporter activity and hand waving that the enabler would be the nuclear pore. We need to discuss whether this is ok. If not, we have a biological process called 'import into the nucleus'. I think that process is a better asserted match for the black box reaction below that the regulation BP. I will put it on the agenda for a future call.

deustp01 commented 3 months ago

I think that process is a better asserted match for the black box reaction below that the regulation BP.

Questions, perhaps to set up that weeds discussion

ukemi commented 3 months ago

You're not missing anything. I would argue for the import process assertion. I think we are seeing over and over that the inferred transporter activities are often problematic. It was a nice idea, but there are lots of exceptions where what is represented is actually a process. I still think we should discuss it with the other curators.

deustp01 commented 3 months ago

I still think we should discuss it with the other curators.

Definitely - exactly as for the transcriptional regulation we want to come up with a broadly useful annotation standard.