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 397: hemin ID choice #81

Closed
colleenXu opened this issue Jun 28, 2024 · 2 comments
Closed

Asset 397: hemin ID choice #81

colleenXu opened this issue Jun 28, 2024 · 2 comments

Comments

@colleenXu
Copy link

colleenXu commented Jun 28, 2024

This asset currently uses PUBCHEM.COMPOUND:26945as the expected answer ID for "hemin". However, the NodeNorm labels for this entity's IDs are "protoheme".

Instead, I see "hemin" in the NodeNorm conflated-ID labels for hematin PUBCHEM.COMPOUND:455658. BTE consistently has hematin in its result set (often too low for a TopAnswer, but still).

Should this test use this different ID for hemin? Or is this a NodeNorm issue?

@colleenXu
Copy link
Author

Related to other issues:

@colleenXu
Copy link
Author

Added to #62 (comment) to consolidate issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant