Seven or eight valence atoms with Dative bonds in the ring cause performance degradation for Add/remove hydrogens operation
- Dominant language
- C++
- Stars
- 406
- Forks
- 134
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 24
Description
**Steps to Reproduce**
1. Load from file: [Seven or eight valence atoms with Dative bonds in the ring cause performance degradation for Add-remove hydrogens operation.zip](https://github.com/epam/Indigo/files/14627921/Seven.or.eight.valence.atoms.with.Dative.bonds.in.the.ring.cause.performance.degradation.for.Add-remove.hydrogens.operation.zip)
2. Press Add/Remove implicit hydrogens button

**Actual behavior**
System turns into infinite calculation
(for Valence=7 it took 76 seconds to calculate)
But if you delete one bond (i.e. make it not cycle) - it works very fast
NOTE: For Ring Topology of bonds - it works fast!
NOTE: For Hydrogen bonds - same problem - works extremely slow (but with ring topology - fast)
**Expected behavior**
It works as fast as not cycled molecule with same number of atoms

**Ketcher version** [e.g. v2.4.2].
- Indigo Toolkit Version 1.18.0.0-g73c32dec3-wasm32-wasm-clang-12.0.0
- Ketcher Version 2.19.0-rc.2 Build at 2024-03-11; 12:30:30
- Chrome Version Version 122.0.6261.112 (Official Build) (64-bit)
- Win10
- Issue found while testing https://github.com/epam/Indigo/issues/1436
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.