NCATSTranslator / testing

Materials and tools for testing Translator components
1 stars 9 forks source link

ChemicalEntity <derivesFrom> ChemicalEntity (PUBCHEM.COMPOUND:6140 - L-phenylalanine) #169

Open sstemann opened 2 years ago

sstemann commented 2 years ago

Query: derivesFrom.json PK: 15f5935e-4479-405c-9e4b-f0cda7194df7 Control: Looking for what metabolite (chemicalEntity ) derives from L-phenylalanine. The control should be L-tyrosine (Tyrosine also fine)

image

finnagin commented 2 years ago

Hey @MarkDWilliams , just as an fyi I made a workflowatized version of this query you could try looking at. It basically takes the query from above but overlays normalized google distance as well as fisher exact test values over the edges. It then scores and ranks the results using those values.

Unfortunately, it cannot currently be run through the workflow runner since @kennethmorton is still hard at work getting its fanning and merging functionality implemented. However, you can still run it through ARAX for the time being.

MarkDWilliams commented 2 years ago

Expected result

Subject Predicate Object
PUBCHEM.COMPOUND:6057 biolink:derives_from PUBCHEM.COMPOUND:6140
webyrd commented 2 years ago

Thanks, Sarah and Mark!

Sorry I missed the call today. I'll look into why Unsecret didn't return any answers.

Cheers,

--Will

On Tue, Jan 25, 2022 at 12:00 PM MarkDWilliams @.***> wrote:

Expected result Subject Predicate Object PUBCHEM.COMPOUND:6057 biolink:derives_from PUBCHEM.COMPOUND:6140

— Reply to this email directly, view it on GitHub https://github.com/NCATSTranslator/testing/issues/169#issuecomment-1021406039, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI5HBGTFJFDSAXXD3QXVKLUX3JJ7ANCNFSM5MWQW6LQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were assigned.Message ID: @.***>

colleenXu commented 2 years ago

To update: we have the data from SEMMEDDB. However, there are issues retrieving it.

There seems to be a semantic + ID-mapping mismatch between:

cbizon commented 2 years ago

Hi @colleenXu can you make an issue for the semantic types here: https://github.com/TranslatorSRI/NodeNormalization/issues

FWIW, if the NN matched the biolink model mappings (which we will revisit), that won't allow there will still be a mismatch to the query. There's a mixin, I think, that would allow chemicals + proteins.

vdancik commented 2 years ago

@cbizon , @sierra-moxon , is there any appetite to bring Polypeptide back to chemical hierarchy?