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

Fix the structure appearance in the final geometry tab. #92

Merged
merged 3 commits into from
Jun 30, 2021

Conversation

yakutovicha
Copy link
Member

fixes #69

@yakutovicha yakutovicha requested review from mbercx and csadorf June 29, 2021 12:50
Comment on lines 228 to 230
self.struct_view._viewer.handle_resize()
self.struct_view._viewer.camera = "perspective"
self.struct_view._viewer.camera = "orthographic"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Do you need all three of these changes to trigger the rendering or would either the first or the last two suffice?
  • What happens if the camera was previously set to "perspective"?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you need all three of these changes to trigger the rendering or would either the first or the last two suffice?

I couldn't entirely figure out the origin of the problem. The solution suggested here didn't work as-is. By pure luck, I discovered that changing the camera mode would let the structure appear. Therefore, I always switch camera to the non-default "perspective" mode and back to the default "orthographic".

What happens if the camera was previously set to "perspective"?

It is changed back to "orthographic".

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tested this, all commands are necessary for the fix to work. I'm going to submit a minor patch that will keep the user set camera when switching between tabs, otherwise this is good.

@csadorf csadorf removed the request for review from mbercx June 30, 2021 15:46
@csadorf csadorf merged commit 13f0bc4 into develop Jun 30, 2021
@yakutovicha yakutovicha deleted the fix/optimized-structure-appearance branch June 30, 2021 17:27
csadorf added a commit that referenced this pull request Jul 14, 2021
Co-authored-by: Carl Simon Adorf <simon.adorf@epfl.ch>
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.

🐛 FIX: Structure not showing in the "Final Geometry" tab of the outputs.
2 participants