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

Bad drawing of end points for dative bonds #5943

Closed
DavidACosgrove opened this issue Jan 7, 2023 · 1 comment
Closed

Bad drawing of end points for dative bonds #5943

DavidACosgrove opened this issue Jan 7, 2023 · 1 comment
Labels
Milestone

Comments

@DavidACosgrove
Copy link
Collaborator

Describe the bug
When drawing the ferrocene in the V3000 below, the ellipses in the SVG marking the end points of the dative bonds are very large and basically wrong.

To Reproduce
'''
ferrocene
ChemDraw01072310592D

0 0 0 0 0 0 V3000
M V30 BEGIN CTAB
M V30 COUNTS 15 14 0 0 0
M V30 BEGIN ATOM
M V30 1 C 0.619616 1.206807 0.000000 0 CHG=-1
M V30 2 C 0.211483 1.768553 0.000000 0
M V30 3 C -1.283936 1.861329 0.000000 0
M V30 4 C -1.796429 1.358429 0.000000 0
M V30 5 C -0.634726 0.966480 0.000000 0
M V30 6 C 0.654379 -1.415344 0.000000 0 CHG=-1
M V30 7 C 0.249886 -0.858607 0.000000 0
M V30 8 C -1.232145 -0.766661 0.000000 0
M V30 9 C -1.740121 -1.265073 0.000000 0
M V30 10 C -0.580425 -1.662922 0.000000 0
M V30 11 C 1.759743 0.755930 0.000000 0
M V30 12 C 1.796429 -1.861329 0.000000 0
M V30 13 Fe -0.554442 0.032137 0.000000 0
M V30 14 * -0.601210 1.478619 0.000000 0
M V30 15 * -0.537835 -1.172363 0.000000 0
M V30 END ATOM
M V30 BEGIN BOND
M V30 1 1 1 5
M V30 2 2 4 5
M V30 3 1 4 3
M V30 4 2 2 3
M V30 5 1 1 2
M V30 6 1 6 10
M V30 7 2 9 10
M V30 8 1 9 8
M V30 9 2 7 8
M V30 10 1 6 7
M V30 11 1 1 11
M V30 12 1 6 12
M V30 13 9 14 13 ENDPTS=(5 1 2 3 4 5) ATTACH=ANY
M V30 14 9 15 13 ENDPTS=(5 9 10 7 8 6) ATTACH=ANY
M V30 END BOND
M V30 END CTAB
M END
'''

Expected behavior
Small circles round atoms 1-5, 6-10.

Screenshots
image

Configuration (please complete the following information):
Master branch, built from scratch on macOS.

Additional context
I will now go and fix it.

@greglandrum
Copy link
Member

Fixed in #5946

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

No branches or pull requests

2 participants