epam / epam/Indigo

Adding hydrogens doesn't work for ordinary bonds wrapped to Multipal group type S-Group

Open
#1,626 2 comments 0 reactions 1 assignee Claimed by @AliaksandrDziarkach View on GitHub
Priority: High 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**
Adding hydrogens doesn't work for ordinary bonds wrapped to Multipal group type S-Group
![image](https://github.com/epam/Indigo/assets/26869421/332045ed-fd10-40c0-ac36-28f6db602996)

**Steps to Reproduce**
1. Load from file: [Adding hydrogens doesn't work for ordinary bonds wrapped to Multipal group type S-Group.zip](https://github.com/epam/Indigo/files/14077300/Adding.hydrogens.doesn.t.work.for.ordinary.bonds.wrapped.to.Multipal.group.type.S-Group.zip) (unzip first)
3. Press Add/Remove explicit hydrogens button

**Actual behavior**
System throws exception: IndigoException: molecule: The group is already collapsed or invalid
![image](https://github.com/epam/Indigo/assets/26869421/785dfc6d-b373-435e-8fcc-08fdc7c4c5f2)
![image](https://github.com/epam/Indigo/assets/26869421/dfa5864d-1b2f-462e-9f18-907e4bccea6a)
[uat.test.lifescience.opensource.epam.com-1706466744453.log](https://github.com/epam/Indigo/files/14077301/uat.test.lifescience.opensource.epam.com-1706466744453.log)

**Expected behavior**
No exception happens

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.