Skip to content

#8363 – Preserve tail bond length after Arrange as a Ring#3450

Closed
Copilot wants to merge 5 commits intomasterfrom
copilot/fix-bond-length-issue
Closed

#8363 – Preserve tail bond length after Arrange as a Ring#3450
Copilot wants to merge 5 commits intomasterfrom
copilot/fix-bond-length-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 16, 2026

  • Run black formatting check on arrange_ring_tail_length.py (black missing, installed)
  • Apply black formatting to arrange_ring_tail_length.py
  • Run targeted arrange_ring_tail_length integration test
  • Update HELM string concatenation for clarity
  • Run code review + CodeQL
Original prompt

This section details on the original issue you should resolve

<issue_title>Bond lenth become wrong after Arrange as a Ring option applied</issue_title>
<issue_description>Steps to Reproduce

  1. Open Macromolecules mode - Flex canvas (clean canvas)
  2. Load following HELM: PEPTIDE1{C.C.C.C.C.C.C.C.C.C.C.C.C}$PEPTIDE1,PEPTIDE1,5:R3-8:R3$$$V2.0
Image
  1. Select ring and right tail of the chain
Image
  1. Call context menu and click Arrange as a Ring option

Actual behavior
System layout monomer in the ring as expected but distance (e.g. bond length) between monomer in the tail is less than standard monomer size
Image

Expected behavior
System layout monomer in the ring as expected, distance (e.g. bond length) between monomer has the same standard length
Image

Related issue: epam/ketcher#8363

Environment details:

  • Ketcher Version 3.12.0-rc.1 Build at 2026-01-08; 17:35:48
  • Indigo Version 1.40.0-rc.1.0-g63d8f2068-wasm32-wasm-clang-19.0.0
  • Chrome Version 143.0.7499.170 (Official Build) (64-bit)
  • Win10</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits January 16, 2026 08:09
Co-authored-by: AlexeyGirin <26869421+AlexeyGirin@users.noreply.github.com>
Co-authored-by: AlexeyGirin <26869421+AlexeyGirin@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix bond length issue after applying 'Arrange as a Ring' #8363 – Preserve tail bond length after Arrange as a Ring Jan 16, 2026
Copilot AI requested a review from AlexeyGirin January 16, 2026 08:23
@AlexeyGirin AlexeyGirin marked this pull request as ready for review January 16, 2026 10:54
@AlexeyGirin
Copy link
Copy Markdown
Collaborator

@copilot please format changes in api/tests/integration/tests/layout/arrange_ring_tail_length.py file in black way

Copilot AI and others added 2 commits January 19, 2026 18:31
Co-authored-by: AlexeyGirin <26869421+AlexeyGirin@users.noreply.github.com>
Co-authored-by: AlexeyGirin <26869421+AlexeyGirin@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 19, 2026

@copilot please format changes in api/tests/integration/tests/layout/arrange_ring_tail_length.py file in black way

Applied black formatting to arrange_ring_tail_length.py and kept it formatted after the latest tweaks (commit 582f01e).

@AlexeyGirin AlexeyGirin requested review from AliaksandrDziarkach and removed request for AlexeyGirin January 19, 2026 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bond length become wrong after Arrange as a Ring option applied

2 participants