-
Notifications
You must be signed in to change notification settings - Fork 141
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
Forbid a charged carbene in singlet_carbene_intra_disproportionation #214
Conversation
f7aa4c9
to
731283b
Compare
…nation Forbidding a charged carbene such as in C[C-]=[N+]=O from reacting in the singlet_carbene_intra_disproportionation family. See #213.
731283b
to
056ca9c
Compare
@nyee, thanks for your comment in #213. |
@zjburas , I re-ran Travis, and this PR is now clean. Could you take a look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, every *1 atom has a c0 attribute now and you removed the forbidden group. My only question is if this change (without the forbidden group) still fixes your original problem?
Yes, it does the trick and solves #213 as well. |
Great, then I think someone with the authority can merge this. |
@nyee, I still can't merge PRs on the database repo... strange. |
@alongd, I modified the permissions. You should be able to merge now. |
Thanks @mliu49 ! |
This solves #213 by adding the simple forbidden group
to
carbene in singlet_carbene_intra_disproportionation
.@zjburas and I were thinking what would be the best approach (adding
c0
to all node? leaving the forbidden group?) @nyee would you have any suggestions?