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

Label shift problem for ambiguous monomers #5982

Closed
Tracked by #6601
AlexeyGirin opened this issue Nov 18, 2024 · 1 comment · Fixed by #5985
Closed
Tracked by #6601

Label shift problem for ambiguous monomers #5982

AlexeyGirin opened this issue Nov 18, 2024 · 1 comment · Fixed by #5985

Comments

@AlexeyGirin
Copy link
Collaborator

Steps to Reproduce

  1. Toggle to Macro - Flex mode
  2. Load HELM from paste from clipboard way: PEPTIDE1{[O1[C@@H]%91[C@H](O)[C@H](O%92)[C@H]1CO%93.[*:3]%91.[*:1]%93.[*:2]%92 |$;;;;;;;;;_R3;_R1;_R2$|]}$$$$V2.0
    image
  3. Load one more HELM from paste from clipboard way: RNA1{[O1[C@@H]%91[C@H](O)[C@H](O%92)[C@H]1CO%93.[*:3]%91.[*:1]%93.[*:2]%92 |$;;;;;;;;;_R3;_R1;_R2$|]([C1(C2=C(N=CN=1)N%91C=N2)N.[*:1]%91 |$;;;;;;;;;;_R1$|])[P%91(O)(O)=O.[*:1]%91 |$;;;;_R1$|]}$$$$V2.0

Actual behavior
Sugar label (Mod0) shifted from the center of monomer
image

Expected behavior
Sugar label (Mod0) at center of monomer
image

Versions

  • Ketcher Version 2.25.0-rc.1 Build at 2024-09-03; 15:31:03
  • Indigo Toolkit Version 1.24.0-rc.1.0-gb8f099851-wasm32-wasm-clang-19.0.0
  • Chrome Version 128.0.6613.114 (Official Build) (64-bit)
  • Win10

Reference issues: epam/Indigo#2189

@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Dec 6, 2024

Tested. Bug fixed.
2024-12-06_18h33_59
2024-12-06_18h34_28

Desktop:

  • OS: Windows 11
  • Browser Chrome
  • Version 128.0.6613.120 (Official Build) (64-bit)

Ketcher version
[Version 2.28.0-rc.1]
Indigo version
[Version 1.27.0-rc.1]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants