search

lambdacasserole / avogadrio

Worship your favorite molecule by setting it as your wallpaper.
https://avogadr.io/
MIT License
297 stars 11 forks source link

Deoxyribonucleic acid displayed incorrectly #32

Closed remusmg closed 5 years ago

remusmg commented 5 years ago

If you input "deoxyribonucleic acid", the result is a phenolphtalein molecule with an extra ether group connecting the two phenols ¯_(ツ)_/¯

lambdacasserole commented 5 years ago

Super weird one there, well found! Unfortunately this seems to be an issue with an external service that Avogadrio uses to convert compound names into SMILES for rendering. Try putting “deoxyribonucleic acid” in here to see what I mean: https://cactus.nci.nih.gov/chemical/structure

lambdacasserole commented 5 years ago

Closing this, bug is external to the project.

remusmg commented 5 years ago

I see... So we're just gonna have to deal with it then. I don't even know what I was expecting, because deoxyribonucleic acid is a class of compounds rather than a specific compound. Kinda like putting "amino acid" in. The code wouldn't know exactly what you mean.

Thanks for looking into it though, and lovely website concept/design. Cheers

În lun., 29 apr. 2019 la 16:35 Saul Johnson notifications@github.com a scris:

Super weird one there, well found! Unfortunately this seems to be an issue with an external service that Avogadrio uses to convert compound names into SMILES for rendering. Try putting “deoxyribonucleic acid” in here to see what I mean: https://cactus.nci.nih.gov/chemical/structure

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/lambdacasserole/avogadrio/issues/32#issuecomment-487604626, or mute the thread https://github.com/notifications/unsubscribe-auth/AL6IHSVIY2INREDLWBW4F4LPS4BRNANCNFSM4HJCDWYA .