epam / epam/Indigo

Add/Remove hydrogens ignores existence of R-group connection points if aromatic atom present on the canvas

Open
#1,643 0 comments 0 reactions 1 assignee Claimed by @AliaksandrDziarkach View on GitHub
Priority: Low query features phase II
Dominant language
C++
Stars
406
Forks
134
Avg merge
2d 11h
Merged PRs (30d)
24

Description

**Ketcher version** [e.g. v2.4.2].
- Indigo Toolkit Version 1.18.0-dev.4.0-g085ab345f-x86_64-linux-gnu-11.2.1
- Ketcher Version 2.17.0-rc.2 Build at 2024-01-03; 17:17:22
- Chrome Version 120.0.6099.225 (Official Build) (64-bit)
- Win10
- Issue found while testing https://github.com/epam/Indigo/issues/1436

**Describe the bug**
Add/Remove hydrogens ignores existence of R-group connection points if aromatic atom present on the canvas

**Steps to Reproduce**
1. Load from file: [IgnoringConnectionPoints.zip](https://github.com/epam/Indigo/files/14103439/IgnoringConnectionPoints.zip) (unzip first)
3. Press Add/Remove explicit hydrogens button
![image](https://github.com/epam/Indigo/assets/26869421/c6442349-f9ea-43fc-811a-63736346164a)

**Actual behavior**
System adds hydrogens ignoring connection points
![image](https://github.com/epam/Indigo/assets/26869421/7a713e85-1b44-45dc-956f-5669b73c46bb)

**Expected behavior**
System adds less hydrogens taking into account connection points
![image](https://github.com/epam/Indigo/assets/26869421/af44298c-9cba-445b-9b4a-559df0867960)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.