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

Common atoms loaded as aliphatic in SMARTS mode #1423

Closed
AliaksandrDziarkach opened this issue Dec 4, 2023 · 0 comments · Fixed by #1424
Closed

Common atoms loaded as aliphatic in SMARTS mode #1423

AliaksandrDziarkach opened this issue Dec 4, 2023 · 0 comments · Fixed by #1424
Assignees
Milestone

Comments

@AliaksandrDziarkach
Copy link
Collaborator

Load some common atom in SMILES mode - got exactly this atoms.
Loads same atoms in SMARTS mode - got atoms with 'A' atribute which means "Aliphatic"
For example load "BrI" as SMILES - got "Br-I", loads as SMARTS - got "Br;A-I;A", which is a bug.

According to documentation "only atoms on the following list can be considered aromatic: C, N, O, P, S, As, Se", so Aliphatic has no sence for all other atoms.

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

Successfully merging a pull request may close this issue.

1 participant