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

feat: double plot download resolution #294

Merged
merged 1 commit into from
Apr 13, 2023
Merged

feat: double plot download resolution #294

merged 1 commit into from
Apr 13, 2023

Conversation

ESCRI11
Copy link
Contributor

@ESCRI11 ESCRI11 commented Apr 13, 2023

This closes #291

Description

I updated the pixel size operations to be doubled. That way a plot that by default was downloaded with a resolution of 640x640 now is 1280x1280.

@ESCRI11 ESCRI11 requested a review from ivokwee April 13, 2023 09:54
@ivokwee
Copy link
Member

ivokwee commented Apr 13, 2023

OK for now. We actually have the "res" parameter in the PlotModule that increases resolution like 80dpi or 150dpi. Seems it was not used for plotly export. For other formats it was applied. Let's keep this solution for now.

@ivokwee ivokwee closed this Apr 13, 2023
@ivokwee ivokwee reopened this Apr 13, 2023
@ivokwee ivokwee merged commit 32c7807 into master Apr 13, 2023
@ESCRI11 ESCRI11 deleted the fix-#291 branch April 13, 2023 13:44
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.

plotly exported PNG are low resolution
2 participants