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.

Sign up for GitHub

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

Jump to bottom

unspecified branch bonds in SMARTS don't have aromaticity set #3342

Closed
greglandrum opened this issue Aug 17, 2020 · 0 comments
Closed

unspecified branch bonds in SMARTS don't have aromaticity set #3342

greglandrum opened this issue Aug 17, 2020 · 0 comments
Assignees
@greglandrum
Labels
bug
Milestone
2020_03_6

Comments

@greglandrum
Copy link
Member

Here's a demo:
image

This isn't a giant deal since the bond orders and aromaticity flags in reactions created from SMARTS are there as a convenience to make it easier to translate to SMILES, but it should still be fixed.
@greglandrum greglandrum self-assigned this Aug 17, 2020
@greglandrum greglandrum added this to the 2020_03_6 milestone Aug 17, 2020
greglandrum added a commit to greglandrum/rdkit that referenced this issue Aug 17, 2020
@greglandrum
Fixes rdkit#3342
cff36b4
greglandrum added a commit that referenced this issue Sep 19, 2020
@greglandrum
Fixes #3342 (#3343)
d50c7f8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@greglandrum greglandrum

Labels
bug
Projects
None yet
Milestone
2020_03_6
Development

No branches or pull requests
1 participant
@greglandrum