NCATSTranslator / Feedback

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

Bug: MVP2 Chemical Selector is restricted to the first result only #885

Closed sandrine-muller-research closed 1 month ago

sandrine-muller-research commented 2 months ago

Original: Examples - search Potassium Ion or Potassium chloride (which is a treatment for hypokalemia)

see @andrewsu 's comment here It looks like node norm makes the distinction. But NameRes does not output anything. @gaurav do you have any clue of why this is happening?

gaurav commented 2 months ago

Hi Sandrine! This is because of two things:

  1. NameRes only provides /reverse_lookup for the preferred IDs of cliques. Since the preferred clique with PUBCHEM.COMPOUND:813 is CHEBI:29103, you need to use that when querying NameRes.
  2. When generating the NameRes index, we keep gene-protein conflation off but we turn drug-chemical conflation on (because otherwise autocomplete ends up with a gazillion different variants of "paracetamol"), so yes, in this case NodeNorm and NameRes disagree on the clique. If it would be useful to do NameRes searches with particular conflations turned on or off, please let us know at https://github.com/TranslatorSRI/NameResolution/issues/56 -- but it's unlikely we'd be able to fit it into this year's round of development unless it was very urgent.
sandrine-muller-research commented 2 months ago

Thank you @gaurav ! Not very urgent indeed, I am closing.

sstemann commented 1 month ago

@sandrine-muller-research did you intend to Close this ticket?

sandrine-muller-research commented 1 month ago

@sstemann yes, my understanding was that this is low priority and it will be a "won't fix" type of issue? I am unclear of what "won't fix" issues should become.

sierra-moxon commented 1 month ago

from TAQA:

we should look at why name lookup will only return one result for chemicals.

gaurav commented 1 month ago
dnsmith124 commented 1 month ago

This is the result of a UI bug, which has been resolved in CI and will be deployed to Test early this week. I'll update the ticket once the fix is live in Test

sandrine-muller-research commented 1 month ago

Thank you @dnsmith124 ! I'll retest at the end of this week! Please, do not hesitate if you'd like me to retest other things.

sstemann commented 1 month ago

i'm not sure if this is the same behavior in CI, but based on yesterday's UI deployment to test, I retested this 8/13 1130AM ET

i dont think this is an improvement, given there are nor more forms of molecules, we seem to hit move 0 results if you click the second or third matching option instead of the first

Left is the Test drop-down and right is the Prod drop-down

image

so the choice (and many other related potassium choices) is there in the drop-down

but ARAs all return 0

https://ui.test.transltr.io/main/results?l=Potassium%20Ion&i=CHEMBL.COMPOUND:CHEMBL1233793&t=3&r=0&q=68c47bcc-28d3-4151-8a7d-55e5ee9285d2

image

one of my questions is - do ARAs support Chembl CURIEs here? (in the original template the input CURIE type was not specified but all of the examples are pubchem https://github.com/NCATSTranslator/TranslatorArchitecture/issues/81 - and in potassium we use Chebi)

if the CURIE source doesn't matter to ARAs, is it worthwhile to have all of these forms of potassium in the dropdown?

@dnsmith124 @gprice1129 @sandrine-muller-research @cartmanbeck

this change in the drop-down is not just for potassium - it now includes other forms of small molecules and in a quick test we are getting 0 results more often since we have made these other forms searchable now

image

image

image

image

image

image

dnsmith124 commented 1 month ago

@sstemann for the UI's part, the change was to resolve a bug that was causing only the first result to show up when searching for chemicals. All the results shown in the drop-down now were still available, they just had to be searched for explicitly with the exact matching characters.

sandrine-muller-research commented 1 month ago

@sstemann on on the ARAX UI, when I search for synonyms, the SRI nodeNorm returns the PUBCHEM ID as the preferred one:

image

The ChEBI is in the same clique but I could not find the ChEMBL ID.

sstemann commented 1 month ago

updated the ticket title based on David's comment 8/13

dnsmith124 commented 1 month ago

Now that this ticket is referring specifically to the chemical autocomplete bug that was present in the UI and that issue has been resolved, I believe we can close out this issue.

Closing.