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

The picture is incorrect when image resolution high and bond thickness changing #5703

Closed
AnnaKosol opened this issue Oct 8, 2024 · 5 comments · Fixed by #5822 or #5827
Closed

The picture is incorrect when image resolution high and bond thickness changing #5703

AnnaKosol opened this issue Oct 8, 2024 · 5 comments · Fixed by #5822 or #5827

Comments

@AnnaKosol
Copy link
Collaborator

Summary
Regarding #5610 the options were deleted from request Render on Ketcher site.
Screenshot 2024-10-08 at 11 35 44

However, the issue is reproduced soon.

Steps to Reproduce

  1. add benzene ring to the Canvas
  2. set image resolution High in the Settings
  3. set bond thickness 2.6 pt in the Settings in the Bond section
  4. click on Apply button
  5. save to png (or svg)

Actual behavior
Screenshot 2024-10-08 at 11 44 10

Expected behavior
Screenshot 2024-10-08 at 11 44 57

Environment details:

  • Ketcher Version 2.26.0-rc.1 Build at 2024-10-04; 18:45:04
  • Indigo Version 1.25.0-rc.1.0-g704673fcb-wasm32-wasm-clang-19.0.0
  • MacOs
  • Chrome 129.0.6668.71 (Official Build) (arm64)

Issues found while testing epam/Indigo#2175

@AliaksandrDziarkach
Copy link
Collaborator

AliaksandrDziarkach commented Oct 9, 2024

Looks like this is ketcher issue:
I use next settings
image
With this settings bond width should be 600ppi2.61/72''=21.6(6)px which is more than one half of bond length which is 40px
But bonds are too thin.

@AliaksandrDziarkach AliaksandrDziarkach transferred this issue from epam/Indigo Oct 9, 2024
@AliaksandrDziarkach AliaksandrDziarkach removed their assignment Oct 9, 2024
@AlexeyGirin
Copy link
Collaborator

AlexeyGirin commented Oct 9, 2024

Decided:
Add hover tooltip (i) with comment that that option applicable to PNG pictures renderer

@AlexeyGirin
Copy link
Collaborator

Not fixed.
( i ) tooltop button was not found

  • Ketcher Version 2.26.0-rc.4 Build at 2024-10-18; 17:00:39
  • Indigo Toolkit Version 1.25.0-rc.3.0-g795119115-wasm32-wasm-clang-19.0.0
  • Chrome Version 130.0.6723.59 (Official Build) (64-bit)
  • Win10

@AlexeyGirin
Copy link
Collaborator

AlexeyGirin commented Oct 22, 2024

Closed. Follow up feature has been created #5842

@AlexeyGirin
Copy link
Collaborator

Verified.

  • Ketcher Version 2.26.0-rc.6 Build at 2024-10-25; 09:57:12
  • Indigo Toolkit Version 1.25.0-rc.4.0-g8862ccb7e-wasm32-wasm-clang-19.0.0
  • Chrome Version 130.0.6723.59 (Official Build) (64-bit)
  • Win10

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