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

Update qMRMLScreenShotDialog to support high-resolution magnified screenshot #3885

Open
slicerbot opened this issue Mar 13, 2020 · 7 comments
Milestone

Comments

@slicerbot
Copy link
Collaborator

This issue was created automatically from an original Mantis Issue. Further discussion may take place here.

@lassoan lassoan added this to the Backlog milestone Mar 16, 2020
@lassoan lassoan removed the backlog label Mar 16, 2020
@lassoan
Copy link
Contributor

lassoan commented Oct 15, 2020

Scale factor is available in the annotation screenshot dialog, but setting >1 value corrupts the rendering:

image

(Slicer-4.11.20200930 on Windows, Intel integrated graphics)

@pieper
Copy link
Member

pieper commented Oct 15, 2020

Did you try both CPU and GPU volume rendering?

@lassoan
Copy link
Contributor

lassoan commented Oct 15, 2020

Higher-resolution rendering fails the same way with CPU and GPU volume rendering (looks like in the screenshot above).

@pieper
Copy link
Member

pieper commented Oct 15, 2020

Interesting, I believe CPU used to work. It looks like a row-length issue. I wonder if we can snap the row to a multiple of 8 or something similar (just an even number?) as a workaround.

@lassoan
Copy link
Contributor

lassoan commented Feb 11, 2021

Full layout capture works, but it is just "digital zoom" (image is rendered at screen resolution and then it is resized to the requested size), so it does not improve image quality.

We can give it a try after the VTK9 upgrade. Maybe the issue is resolved in VTK and if not then we can report it to VTK developers.

@lassoan
Copy link
Contributor

lassoan commented Feb 11, 2021

If the problem gets fixed then it would be useful to use it also in ScreenCapture module. That would also make it easily accessible in Python (as requested here).

@pieper
Copy link
Member

pieper commented Apr 24, 2023

This came up again on discourse: https://discourse.slicer.org/t/annotation-screenshot-scale-factor/29091/2

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

No branches or pull requests

3 participants