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

Implicit hydrogens for aromatised structures containing Sulphur #3554

Closed
AlexanderSavelyev opened this issue Nov 8, 2023 · 1 comment · Fixed by #3570
Closed

Implicit hydrogens for aromatised structures containing Sulphur #3554

AlexanderSavelyev opened this issue Nov 8, 2023 · 1 comment · Fixed by #3570
Assignees
@Nitvex
Labels
bug
Milestone
Ketcher 2.17.0-rc.1

Comments

@AlexanderSavelyev
Copy link
Contributor

AlexanderSavelyev commented Nov 8, 2023 •
edited
Loading

Steps to Reproduce

  1. Load structure S1C=CN=C1
  2. Click aromatize

Actual behavior
Additional implicit hydrogen appears for Sulfur
image

Expected behavior
No implicit hydrogen should appear for Sulfur inside aromatized rings
@AlexanderSavelyev AlexanderSavelyev added this to the Ketcher 2.17.0-rc.1 milestone Nov 8, 2023
Nitvex added a commit that referenced this issue Nov 14, 2023
@Nitvex
#3554 – Implicit hydrogens for aromatised structures containing Sulphur
df6d75c
@Nitvex Nitvex mentioned this issue Nov 14, 2023
Merged
9 tasks
Nitvex added a commit that referenced this issue Nov 14, 2023
@Nitvex
#3554 – Implicit hydrogens for aromatised structures containing Sulphur
ac1b884
@y-holik
Copy link
Collaborator

y-holik commented Nov 15, 2023

tested and approved

Nitvex added a commit that referenced this issue Nov 15, 2023
@Nitvex
#3554 – Implicit hydrogens for aromatised structures containing Sulph…
69872e2 
…ur (#3570)

* #3554 – Implicit hydrogens for aromatised structures containing Sulphur

* #3554 – Implicit hydrogens for aromatised structures containing Sulphur
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Nitvex Nitvex

Labels
bug
Projects
Ketcher
Status: Done
Milestone
Ketcher 2.17.0-rc.1
Development

Successfully merging a pull request may close this issue.

#3554 – Implicit hydrogens for aromatised structures containing Sulphur epam/ketcher
3 participants
@AlexanderSavelyev @Nitvex @y-holik