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

Removing peptide from sense/antisence chain cause unnessussary phosphate removal #6471

Closed
Tracked by #6602
AlexeyGirin opened this issue Feb 10, 2025 · 1 comment · Fixed by #6486 or #6490
Closed
Tracked by #6602
Assignees
Labels
Antisense Bucket: Antisense related issues Priority: High Sequence mode Bucket: Sequence mode related issues Severity: High High level visibility to the user

Comments

@AlexeyGirin
Copy link
Collaborator

Steps to Reproduce

  1. Go to Macro - Sequence mode (clean canvas)
  2. Load from HELM: RNA1{R(A)P}|RNA2{R(U)P.R(U)}|PEPTIDE1{E.E}|RNA3{R(A)}$RNA1,RNA2,2:pair-2:pair|RNA1,PEPTIDE1,3:R2-1:R1|PEPTIDE1,RNA3,2:R2-1:R1|RNA3,RNA2,2:pair-5:pair$$$V2.0
    Image
  3. Click between E symbols to switch to Edit mode
  4. Press Backspace key to delete left E

Actual behavior
System turns left nucleotide into nucleoside
Image
Image

Expected behavior
System turns remain nucleotide as it is (E is deleted)
Image
Image

Versions

  • Ketcher Version 3.1.0-rc.2 Build at 2025-02-07; 15:09:23
  • Indigo Version 1.29.0-rc.1.0-g2e30df8f7-wasm32-wasm-clang-19.0.0
  • Chrome Version 133.0.6943.54 (Official Build) (64-bit)
  • Win10

Issue found while testing: #5942

@AlexeyGirin AlexeyGirin added Antisense Bucket: Antisense related issues Priority: High Sequence mode Bucket: Sequence mode related issues Severity: High High level visibility to the user labels Feb 10, 2025
@AlexeyGirin AlexeyGirin added this to the Ketcher 3.1.0-rc.3 milestone Feb 10, 2025
@rrodionov91 rrodionov91 self-assigned this Feb 11, 2025
rrodionov91 added a commit that referenced this issue Feb 11, 2025
rrodionov91 added a commit that referenced this issue Feb 11, 2025
rrodionov91 added a commit that referenced this issue Feb 11, 2025
…y phosphate removal (#6486)

(cherry picked from commit 2faa694)
rrodionov91 added a commit that referenced this issue Feb 11, 2025
…y phosphate removal (#6486)

(cherry picked from commit 2faa694)
@AlexeyGirin
Copy link
Collaborator Author

AlexeyGirin commented Feb 12, 2025

Verified.
Image
Image

  • Ketcher Version 3.1.0-rc.3 Build at 2025-02-11; 20:29:49
  • Indigo Version 1.29.0-rc.1.0-g2e30df8f7-wasm32-wasm-clang-19.0.0
  • Chrome Version 133.0.6943.60 (Official Build) (64-bit)
  • Win10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Antisense Bucket: Antisense related issues Priority: High Sequence mode Bucket: Sequence mode related issues Severity: High High level visibility to the user
Projects
None yet
2 participants