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.

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

Sysmem should throw an error if we try to connect monomer to already occopied attachment point #2078

Closed
AlexeyGirin opened this issue Jul 3, 2024 · 3 comments · Fixed by #2284
Assignees
Labels
Bug HELM Bucket: Bugs related to Export to HELM format Priority: Medium

Comments

@AlexeyGirin
Copy link
Collaborator

Steps to Reproduce

  1. Go to Macromolecules mode - Flex mode
  2. Load using paste from clipboard IDT type: CHEM1{[MCC]}|RNA1{R(U)P}$CHEM1,RNA1,1:R1-1:R2$$$V2.0
    image

Actual behavior
System load something
image

Expected behavior
Error should occur - Can't connect to R2 attachment point of monomer index 1 of RNA1 - it is already occupied

Versions

  • Ketcher Version 2.23.0-rc.1 Build at 2024-07-02; 08:57:41
  • Indigo Toolkit Version 1.22.0-rc.1.0-g2bef0ce6c-wasm32-wasm-clang-19.0.0
  • Windows 10 Pro
  • Chrome Version 126.0.6478.127 (Official Build) (64-bit)

Issue found while testing - #1188

@AlexeyGirin AlexeyGirin added Bug Priority: Medium HELM Bucket: Bugs related to Export to HELM format labels Jul 3, 2024
@AliaksandrDziarkach
Copy link
Collaborator

Planed to fix with variant monomers helm support.

@AlexeyGirin
Copy link
Collaborator Author

Re-CHECK

@split-ends
Copy link
Collaborator

Successfully verified: an error appears with the corresponding message:
image

Environment details:
Ketcher Version: 2.25.0-rc.1 Build at 2024-09-03; 15:31:03
Indigo Version: 1.24.0-rc.1.0-gb8f099851-wasm32-wasm-clang-19.0.0
Chrome Version: Version 128.0.6613.113 (Official Build) (64-bit)
Windows 11 Enterprise

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug HELM Bucket: Bugs related to Export to HELM format Priority: Medium
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants