Change canvas.print_figure default resolution #305

Activity

1. WeatherGod commented on Jun 21, 2011

   

   WeatherGod

   on Jun 21, 2011

   Member

   More actions

   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.
2. pelson commented on Sep 2, 2012

   

   pelson

   on Sep 2, 2012

   Member

   More actions

   Sounds like an easy win... is there a reason we haven't done it already???
3. efiring commented on Jul 22, 2013

   

   efiring

   on Jul 22, 2013

   Member

   More actions

   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.
4. mdboom commented on Jul 22, 2013

   

   mdboom

   on Jul 22, 2013

   Member

   More actions

   Seems reasonable to me.
5. ghost assigned

   mdboom

   on Jul 22, 2013
6. jmehne commented on Dec 9, 2015

   

   jmehne

   on Dec 9, 2015

   Contributor

   More actions

   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)?
7. tacaswell

   modified the milestones: next major release (2.0)This milestone has been deleted, proposed next point release (2.1)This milestone has been deleted on Dec 9, 2015
8. tacaswell commented on Dec 9, 2015

   

   tacaswell

   on Dec 9, 2015

   Member

   More actions

   This should be folded in with the 2.0 style changes.
9. Tillsten commented on Dec 9, 2015

   

   Tillsten

   on Dec 9, 2015

   Contributor

   More actions

   While on it, i really dislike the use of savefig.dpi in ipython-inline, it should just use the figure.dpi.
10. Tillsten commented on Dec 9, 2015

    

    Tillsten

    on Dec 9, 2015

    Contributor

    More actions

    It should also stop using subplot adjust.
11. tacaswell commented on Dec 9, 2015

    

    tacaswell

    on Dec 9, 2015

    Member

    More actions

    @Tillsten You need to take that up with the IPython folks, mpl does not control that backend.
12. Tillsten commented on Dec 9, 2015

    

    Tillsten

    on Dec 9, 2015

    Contributor

    More actions

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

    added 4 commits that reference this issue on Dec 10, 2015

    Change `savefig.dpi` and `figure.dpi` defaults

    ...

    This issue will close once commit 29b99cc is merged into the 'master' branch.

    29b99cc

    Change `savefig.dpi` and `figure.dpi` defaults

    ...

    This issue will close once commit a40dd9d is merged into the 'master' branch.

    a40dd9d

    Change `savefig.dpi` and `figure.dpi` defaults

    ...

    This issue will close once commit 3f5118c is merged into the 'master' branch.

    3f5118c

    Change `savefig.dpi` and `figure.dpi` defaults

    ...

    This issue will close once commit 016406a is merged into the 'master' branch.

    016406a
14. efiring

    closed this as completedin 016406aon Dec 13, 2015
15. efiring

    added a commit that references this issue on Dec 13, 2015

    Change `savefig.dpi` and `figure.dpi` defaults

    ...

    This issue will close once commit a903315 is merged into the 'main' branch.

    a903315