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.

Sign up for GitHub

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

Jump to bottom

show_colormaps() should check how much display room is available #1781

Closed
ehsteve opened this issue May 18, 2016 · 5 comments
Closed

show_colormaps() should check how much display room is available #1781

ehsteve opened this issue May 18, 2016 · 5 comments
Labels
Effort Low Requires a small time investment Package Novice Requires little knowledge of the internal structure of SunPy Priority Medium Non-urgent action required

Comments

@ehsteve
Copy link
Member

ehsteve commented May 18, 2016

It can generate plots which are larger than my screen so I cannot see all of the colormaps.
@larrymanley
Copy link
Contributor

Is there a mechanism in matplotlib for that? Perhaps use a figure with explicit values for size and dpi. Otherwise, even on a similar screen, the figure could display differently if the user has different defaults specified in .matplotlibrc. Should also remove the live x,y values at the bottom of the plot window.

@larrymanley
Copy link
Contributor

The figure size is set as 5,10 on line 183 of cm.py, just add dpi=___. 64 dpi is conservative and the window can be resized.

@Cadair Cadair added Package Novice Requires little knowledge of the internal structure of SunPy Priority Medium Non-urgent action required Effort Low Requires a small time investment labels May 18, 2016
@abit2
Copy link
Member

abit2 commented Aug 31, 2016 •
edited

@Cadair @ehsteve #1895 please review
new here

@abit2
Copy link
Member

abit2 commented Sep 15, 2016

#1901
@dpshelio
thanks for helping me 😄

@abit2 abit2 mentioned this issue Jan 31, 2017
Merged
@Cadair Cadair closed this as completed Jan 31, 2017
@prateekiiest
Copy link
Contributor

Pls review #1980

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Effort Low Requires a small time investment Package Novice Requires little knowledge of the internal structure of SunPy Priority Medium Non-urgent action required
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@ehsteve @larrymanley @Cadair @abit2 @prateekiiest