Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
sandrine-muller-research opened this issue Jul 26, 2024 · 13 comments
Assignees
Labels
bug Something isn't working Fugu (Sprint 4) - due July 19 in CI This ticket will be fixed in CI by the end of Fugu/Sprint 4 (July 19)

Comments

@sandrine-muller-research
Copy link
Collaborator

sandrine-muller-research commented Jul 26, 2024

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
Copy link

gaurav commented Jul 26, 2024

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 Add support for conflation into NameRes TranslatorSRI/NameResolution#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
Copy link
Collaborator Author

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

@sstemann
Copy link

sstemann commented Aug 5, 2024

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

@sandrine-muller-research
Copy link
Collaborator Author

@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
Copy link
Member

from TAQA:

  • with chemical conflation turned on, we won't have ions to search by?
    • multiple ions in name lookup and still findable
    • potassium as an ion vs a cation
    • we have one in NN but not the other
    • when we do chemical searches, we only show the top result (name lookup returns multiple results, in the dropdown we only get to select the top result)
    • we could investigate why only the top result is showing up in the UI
  • Na, K, etc, none can be returned
  • we do want to search by ions
  • these are too specific -- potassium ion vs. potassium salt -- we need it to be generalized here in the search or the result.
  • when we search for sodium we only have sodium tetradecyl sulfate

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

@sierra-moxon sierra-moxon added Fugu (Sprint 4) - due July 19 in CI This ticket will be fixed in CI by the end of Fugu/Sprint 4 (July 19) and removed Next Phase labels Aug 9, 2024
@gaurav
Copy link

gaurav commented Aug 9, 2024

  • We might also want to conflate e.g. "Potassium cation" with "Potassium" so that there are fewer non-specific results.

@dnsmith124
Copy link
Collaborator

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
Copy link
Collaborator Author

sandrine-muller-research commented Aug 12, 2024

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
Copy link

sstemann commented Aug 13, 2024

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 NCATSTranslator/TranslatorArchitecture#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
Copy link
Collaborator

@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
Copy link
Collaborator Author

@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 sstemann changed the title not able to search for potassium ion anymore Bug: MVP2 Chemical Selector is restricted to the firstr result only Aug 14, 2024
@sstemann sstemann changed the title Bug: MVP2 Chemical Selector is restricted to the firstr result only Bug: MVP2 Chemical Selector is restricted to the first result only Aug 14, 2024
@sstemann
Copy link

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

@dnsmith124
Copy link
Collaborator

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Fugu (Sprint 4) - due July 19 in CI This ticket will be fixed in CI by the end of Fugu/Sprint 4 (July 19)
Projects
None yet
Development

No branches or pull requests

6 participants