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

3d mol files appear as 2D in Miew #3501

Closed
dondaley opened this issue Oct 25, 2023 · 4 comments · Fixed by #4529
Closed

3d mol files appear as 2D in Miew #3501

dondaley opened this issue Oct 25, 2023 · 4 comments · Fixed by #4529
Assignees
Labels

Comments

@dondaley
Copy link

Steps to Reproduce

  1. import 3D mol file'
  2. Click on 3D view
  3. View structure

Actual behavior
3d mol files appear as 2D in Miew
Expected behavior
#D appeared with 3D mol file in previous versions

Screenshots
image

image
image

squashes file into 2D and changes cooridinates
image

and V3000 does not work

Desktop (please complete the following information):

  • OS: Macos 14.0
  • Browser safari
  • Version 17.0

Ketcher version [e.g. v2.4.2].
2.11.0

Test case

Additional context

@mikaukora
Copy link

Based on my testing this happens at d4981dc for the first time.
The commit is included in release/2.18.

Steps to reproduce:

  • Open structure library
  • Select 3D Templates
  • Select fullerene C60 and place it on canvas
  • Select 3D viewer
  • The structure is rendered flat (2d) instead of 3d

mtreinik added a commit to mtreinik/ketcher that referenced this issue Apr 24, 2024
- Rotation was missing z coordinate when switching into chemistry coordinate system.
@mtreinik
Copy link
Contributor

mtreinik commented Apr 24, 2024

The pull request below fixes this issue. The function rotateCoordAxisBy180Degrees was missing z coordinate, thus flattening the molecule to a two-dimensional figure.

NataliaLoginova added a commit that referenced this issue Apr 25, 2024
#3501 3d mol files appear as 2D in Miew
@NataliaLoginova
Copy link
Collaborator

@mtreinik thank you a lot for for fixing the issue #3501!

@rrodionov91 rrodionov91 linked a pull request Apr 26, 2024 that will close this issue
9 tasks
@rrodionov91 rrodionov91 added this to the Ketcher 2.22.0-rc.1 milestone Apr 26, 2024
@NadezhdaPeskun
Copy link

Verified as fixed for CI of Ketcher 2.22.rc1.
ketcher (31).zip

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

Successfully merging a pull request may close this issue.

7 participants