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

Asset 367: wrong answer ID for Bupropion #77

Closed
colleenXu opened this issue Jun 28, 2024 · 2 comments · Fixed by #97
Closed

Asset 367: wrong answer ID for Bupropion #77

colleenXu opened this issue Jun 28, 2024 · 2 comments · Fixed by #97
Labels
Bad Asset Curie The given asset curie is not correct

Comments

@colleenXu
Copy link

colleenXu commented Jun 28, 2024

This asset currently uses PUBCHEM.COMPOUND:3031065 as the expected answer ID for Bupropion. But this chemical appears to have a different structure than Bupropion (the chlorine on its ring is in a diff position compared to Bupropion). I also think the NodeNorm entry for this ID seems odd: none of the ID labels are bupropion (Dev NodeNorm).

Instead, there's another entity with the name Bupropion and lots of equivalent IDs, including the PUBCHEM.COMPOUND:444 that I referenced above: CHEBI:3219.
This shows up in BTE's results with a high score/rank (top 10%).

I think the answer ID for this test needs to be changed.

@colleenXu colleenXu changed the title Asset 367: Bupropion answer ID choice Asset 367: wrong answer ID for Bupropion Jul 2, 2024
@colleenXu
Copy link
Author

Updated issue: figured out that the wrong ID is being used, so this isn't a synonymization issue.

@maximusunc
Copy link
Collaborator

Confirmed CHEBI:3219 with Name Resolver.

@maximusunc maximusunc added the Bad Asset Curie The given asset curie is not correct label Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bad Asset Curie The given asset curie is not correct
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants