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

ball and stick representation is missing some sticks #150

Closed
arouinfar opened this issue Jan 28, 2020 · 8 comments
Closed

ball and stick representation is missing some sticks #150

arouinfar opened this issue Jan 28, 2020 · 8 comments

Comments

@arouinfar
Copy link
Contributor

@Denz1994 looks like there is a missing bond between the oxygen and hydrogen in this structure. Can you investigate?
Screen Shot 2020-01-28 at 10 49 11 AM

I'm fairly certain this structure would be an ion, but I don't think that should affect the O-H bond. I'll dig around a bit more and see if I can find any other examples of this.

@arouinfar
Copy link
Contributor Author

I'm fairly certain this structure would be an ion, but I don't think that should affect the O-H bond

Actually, this may be the crux of the problem. I don't think we should be naming ions. It has a 3D button because it's a named thing, but perhaps it shouldn't be named at all.

@arouinfar
Copy link
Contributor Author

Hydroxymethanone doesn't have any direct hits on pubchem, but searching through various CHO2 structures, I think the closest match is hydroxycarbonyl which is a radical. Radicals have an unpaired electron, but no net charge.

@arouinfar
Copy link
Contributor Author

Here's another problematic case, which appears to be a formyl radical.
image

@arouinfar
Copy link
Contributor Author

For comparison, here are the Java structures.
Screen Shot 2020-01-28 at 11 26 37 AM
Screen Shot 2020-01-28 at 11 27 12 AM

@Denz1994
Copy link
Contributor

After talking this over with @arouinfar, it looks like this requires a larger discussion. Currently, we are displaying both ions and radicals. This behavior matches what is done in Java, but we should decide if we should either not show ions/radicals. Or do nothing, with the understanding that no bond assumes either a ion or radical (perhaps noting in teacher tips)

If we choose to not show ions/radicals, the question becomes: How do we identify this from the PubChem data set? Could we use total charge or general formula? I would not advocating for this option, because it may require pulling new data from PubChem. Also, we don't have a way to determine the total charge at the moment.

Labeling for design meeting for input.

@arouinfar
Copy link
Contributor Author

arouinfar commented Jan 30, 2020

I think the problem here is solely the radicals. There appear to be several in otherMoleculeData.js which was pulled from PubChem in 2013. Examples include:

I'm not sure why the names in the sim all vary from what's on PubChem, but perhaps it's because we pulled the data from PubChem so long ago.

Looking at the PubChem entries for any of these radicals, you'll see that the 3D structure isn't rendered. (The same is not true for ions such as nitrate. However, if you build NO3 in the sim, it will not be given a name/3D dialog.)
image

My best guess is that the ball and stick representation is not appropriate for radicals, which is what leads to the missing sticks in the screenshots earlier in this issue.

It would be great if we could somehow remove radicals from otherMoleculeData.js so they are not named (and therefore given a 3D dialog), but I wouldn't even begin to know how to find them in the 9000+ compounds in the database.

Since the current behavior matches Java and there haven't been any user complaints about this particular issue, I'm inclined to leave things the way they are and document this in the teacher tips.

@BLFiedler can you review this issue and let me know what you think?

@brettfiedler
Copy link
Member

brettfiedler commented Feb 4, 2020

I think leaving the current behavior and making a note in the teacher tips is probably the best course of action.

Is there any way to isolate the radicals in Pubchem based on the fact that they don't tend to have 3D structures? Not sure what other criteria we can work with, But yeah, ball and stick without some sort of accompanying notation (lewis dot structure) is not really a useful view for radicals. I would suspect they are outside of the learning goals of this sim as well?

Can you clarify the behavior for ions? Do they display 3D structures or no or sometimes?

@arouinfar
Copy link
Contributor Author

I think leaving the current behavior and making a note in the teacher tips is probably the best course of action.

Thanks @BLFiedler! I'll add this to the Teacher Tips issue #152, and we can close this one.

Is there any way to isolate the radicals in Pubchem based on the fact that they don't tend to have 3D structures?

We think it might be possible to remove the radicals if we pull all the data from PubChem again, but it's a bit complicated. In last week's design meeting, we decided that this could be a project for a visiting/intern developer, but it's not something we plan to do before the 1.0 release.

Can you clarify the behavior for ions? Do they display 3D structures or no or sometimes?

I tried building several common ions in the sim (such has hydroxide or nitrate), and they were never named so they never got the 3D dialog. When we pulled the data from PubChem, I believe we excluded ions which I think was the right call.

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

No branches or pull requests

3 participants