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

R Group login condition is wrong if loaded from MOL #2699

Open
AlexeyGirin opened this issue Dec 20, 2024 · 0 comments
Open

R Group login condition is wrong if loaded from MOL #2699

AlexeyGirin opened this issue Dec 20, 2024 · 0 comments
Labels
Export to MOL Bucket: Export to MOL related issues Micromolecules canvas Bucket: Micromolecules canvas related issues Mol v2000 Loader Replacement Issues related to replacement Ketcher mol loader with Indigo one Priority: Low regress found by autotest Severity: Low Low level visibility to the user Test Automation affected That issue affect on existed auto test, please update screenshots after fix
Milestone
Indigo-1.29.0-rc.1

Comments

@AlexeyGirin
Copy link
Collaborator

AlexeyGirin commented Dec 20, 2024 •
edited
Loading

Steps to Reproduce

  1. Go to Molecules mode
  2. Load from MOL: markush.zip

Actual behavior
R Group login condition is wrong (>0)
image

Expected behavior
R Group login condition is correct (=1)
image

Additional cases: complex-r-group-structure.mol.zip

Versions

  • Ketcher Version 2.28.0-rc.3 Build at 2024-12-11; 14:32:57
  • Indigo Version 1.27.0-rc.1.0-g8d656a341-wasm32-wasm-clang-19.0.0
  • Chrome Version 131.0.6778.109 (Official Build) (64-bit)
  • Win10
@AlexeyGirin AlexeyGirin added Bug Priority: Low Severity: Low Low level visibility to the user labels Dec 20, 2024
@AlexeyGirin AlexeyGirin transferred this issue from epam/ketcher Dec 20, 2024
@AlexeyGirin AlexeyGirin added the Micromolecules canvas Bucket: Micromolecules canvas related issues label Dec 20, 2024
@AlexeyGirin AlexeyGirin added this to the Indigo-1.28.0-rc.2 milestone Dec 20, 2024
@AlexeyGirin AlexeyGirin added regression Test Automation affected That issue affect on existed auto test, please update screenshots after fix regress found by autotest Export to MOL Bucket: Export to MOL related issues and removed regression labels Dec 20, 2024
@AlexeyGirin AlexeyGirin changed the title R Group login condition is wrong if loaded from MOL [Replace Ketcher mol loader with Indigo] R Group login condition is wrong if loaded from MOL Dec 20, 2024
@AlexeyGirin AlexeyGirin added the Mol v2000 Loader Replacement Issues related to replacement Ketcher mol loader with Indigo one label Jan 13, 2025
@AlexeyGirin AlexeyGirin changed the title [Replace Ketcher mol loader with Indigo] R Group login condition is wrong if loaded from MOL R Group login condition is wrong if loaded from MOL Jan 15, 2025
@AlexeyGirin AlexeyGirin removed the Bug label Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Export to MOL Bucket: Export to MOL related issues Micromolecules canvas Bucket: Micromolecules canvas related issues Mol v2000 Loader Replacement Issues related to replacement Ketcher mol loader with Indigo one Priority: Low regress found by autotest Severity: Low Low level visibility to the user Test Automation affected That issue affect on existed auto test, please update screenshots after fix
Projects
None yet
Milestone
Indigo-1.29.0-rc.1
Development

No branches or pull requests
1 participant
@AlexeyGirin