NCATSTranslator / Feedback

A repo for tracking gaps in Translator data and finding ways to fill them.
7 stars 0 forks source link

Strange logic (or incorrect edge type) for what gene's activity may be decreased by lithium #830

Closed khanspers closed 4 months ago

khanspers commented 4 months ago

https://ui.test.transltr.io/main/results?l=Lithium%20Ion&i=CHEBI:49713&t=4&r=0&q=9b265413-9819-42bb-a913-59b7138a36ad

EPH2B is returned, and the inferred edge looks like this:

EPHB2 - downregulates - AKT1 - has increased activity or abundance caused by - Lithium ion (same pattern for INS and GSK3B)

To me, that logic doesn't say anything about the effect of lithium on EPH2B, which is what the query is asking.

Screen Shot 2024-07-03 at 8 26 06 AM

It looks like the answer is coming from BTE (deduced by filtering for EPHB2 in the UI, then looking at Reasoning Agents in the Filters menu), and in ARAX I see the following support graph:

Screen Shot 2024-07-03 at 8 31 14 AM

Here there are affects/regulates edges from AKT1, INS, and GSK3A to EPHB2. So it seems the downregulates edge from EPH2B to AKT1/INS/GSK3A is just incorrect in the UI and should be something that indicates the interaction is the other direction?

gglusman commented 4 months ago

Related to #468

mikebada commented 4 months ago

I agree; I think if the first assertion in the path were instead EPHB2 downregulated_by AKT1, I think it would be valid...

sierra-moxon commented 4 months ago

@andrewsu - I noticed the result in the screenshot has infores:biothings-explorer tagged. Would you have a chance to take a look at this one?

andrewsu commented 4 months ago

If I'm reading this correctly, I think @khanspers is pointing out that the visualization in the UI (the first screenshot above) does not reflect what BTE is outputting (and visualized by ARAX in the second screenshot above). So I think this is an issue for the UI team to look into? tagging @Genomewide

(@khanspers @gglusman @mikebada please let me know if I'm missing something, or maybe add a 👍reaction if you agree...)

sierra-moxon commented 4 months ago

apologies - yes, you are right. thank you for taking a look.

gglusman commented 4 months ago

I think the main issue here is that by mixing directionality of predicates (as evidenced by the active voice in the 1st "downregulates", and the couched passive voice in the 2nd "...caused by"), the pathway itself is not relevant as an answer to the query. That's the connection to #468 I made above. If we instead used normalized predicate directions and stated that "Lithium ion increases activity or abundance of AKT1", and correspondingly displayed the arrow from right to left, the issue would become visually apparent. But how to display it is not the issue. This pathway logic is producing incorrect results.

andrewsu commented 4 months ago

BTE is reporting two edges for AKT1 -- Lithium - affects / increases - AKT1 and AKT1 - regulates / downregulates - EPHB2 -- both in the "forward direction". This pathway logic seems fine to me, and it looks like the UI has incorrectly reversed the directionality in displaying EPHB2 - downregulates - AKT1. @gglusman are you seeing this differently?

gprice1129 commented 4 months ago

I agree with your assessment @andrewsu. The UI should be displaying is downregulated by or something equivalent.

gprice1129 commented 4 months ago

Found and fixed the bug locally. Will update when pushed to Test.

gprice1129 commented 4 months ago

This is fixed in Test. Closing.