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

Change canvas.print_figure default resolution #305

Closed
ddale opened this issue Jun 20, 2011 · 10 comments
Closed

Change canvas.print_figure default resolution #305

ddale opened this issue Jun 20, 2011 · 10 comments
Assignees
Milestone

Comments

@ddale
Copy link
Contributor

ddale commented Jun 20, 2011

Original report at SourceForge, opened Fri Feb 5 22:59:50 2010

Change canvas.print_figure default resolution from 100 DPI to the resolution of the figure being printed.

@WeatherGod
Copy link
Member

Current behavior that if dpi=None (the default), then defer to the rc param "savefig.dpi" (which seems to default to 100). Maybe we could allow for a string value ("native") that would be tested for and would indicate to use the dpi already set.

@pelson
Copy link
Member

pelson commented Sep 2, 2012

Sounds like an easy win... is there a reason we haven't done it already???

@efiring
Copy link
Member

efiring commented Jul 22, 2013

At the same time, can we change the default figure.dpi to 100? And make the savefig.dpi "native", or whatever ("screen"? "figure"?) be the default there? That way we are not actually changing the default savefig.dpi, which is most likely to be the one that would upset people running automated plot generation, and we are making the figure.dpi more sensible for typical screens--I think.

@mdboom
Copy link
Member

mdboom commented Jul 22, 2013

Seems reasonable to me.

@jmehne
Copy link
Contributor

jmehne commented Dec 9, 2015

Should this issue be closed, or do you still want to change the defaults (savefig.dpi=100 -> savefig.dpi='figure' and figure.dpi=80 -> figure.dpi=100)?

@tacaswell tacaswell modified the milestones: next major release (2.0), proposed next point release (2.1) Dec 9, 2015
@tacaswell
Copy link
Member

This should be folded in with the 2.0 style changes.

@Tillsten
Copy link
Contributor

Tillsten commented Dec 9, 2015

While on it, i really dislike the use of savefig.dpi in ipython-inline, it should just use the figure.dpi.

@Tillsten
Copy link
Contributor

Tillsten commented Dec 9, 2015

It should also stop using subplot adjust.

@tacaswell
Copy link
Member

@Tillsten You need to take that up with the IPython folks, mpl does not control that backend.

@Tillsten
Copy link
Contributor

Tillsten commented Dec 9, 2015

@tacaswell I really think your voice is a lot stronger than the one of a single user.

jmehne added a commit to jmehne/matplotlib that referenced this issue Dec 10, 2015
jmehne added a commit to jmehne/matplotlib that referenced this issue Dec 10, 2015
jmehne added a commit to jmehne/matplotlib that referenced this issue Dec 10, 2015
efiring pushed a commit that referenced this issue Dec 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants