NCATSTranslator / Feedback

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

Were these paths supposed to be merged? #789

Closed sstemann closed 2 months ago

sstemann commented 5 months ago

The node identifiers on the start middle and end nodes look the same, the predicates look the same, the evidence on the predicates look the same

image

https://ui.test.transltr.io/main/results?l=Aceruloplasminemia&i=MONDO:0011426&t=0&r=0&q=77003cf0-f9ff-4864-bc02-b89f2c216c82

dnsmith124 commented 5 months ago

I don't believe these paths should be merged, if you mouse over the nodes that begin with Fech Binds Fe... their names differ after the ellipsis, and they have different curies. For example: Fech Binds Fe2+ To Prin9 To Form Heme (dictyostelium Discoideum) vs Fech Binds Fe2+ To Prin9 To Form Heme (rattus Norvegicus)

I just moused through them and don't see any duplicates.

sierra-moxon commented 3 months ago

from TAQA: [Gaurav] - these would be deemed a cluster using the method I showed in O&O last week 🙂 - [Gwenlyn] see https://github.com/NCATSTranslator/Feedback/issues/863 as example [Chris] - the filter we put on to remove these kinds of pathways should be in TEST via Fugu.

sandrine-muller-research commented 3 months ago

Retested today The nodes in the middle look different but do not seem right/normalized:

image

Result comes from ARAGORN. I went ahead and tried to look at the graphs in ARAX but I get no output:

image

This issue need further retesting at another time.

sstemann commented 3 months ago

i'm not seeing any improvements to these paths returned from ARAGORN in Fugu/Test.

https://ui.test.transltr.io/main/results?l=Aceruloplasminemia&i=MONDO:0011426&t=0&r=0&q=0e83e4a4-0af7-4b6d-8287-53fecd44b8f2

Its the same in CI. Looks like this is not fixed in Fugu or queued up to be deployed to Test. https://ui.ci.transltr.io/main/results?l=Aceruloplasminemia&i=MONDO:0011426&t=0&r=0&q=0886d892-dde0-48f7-847d-e0454e96803c

sstemann commented 2 months ago

it looks like these "FECH binds Fe2+..." nodes are all unique. So I think this is now closed.

https://ui.transltr.io/results?l=Aceruloplasminemia&i=MONDO:0011426&t=0&r=0&q=cf3952b5-8df4-492d-a06d-ff0b09758d15