Skip to content

Conversation

@jenshnielsen
Copy link
Member

Closes #6636

Change default docs dpi for lowr res png to 100 to match default dpi. Remove hires png from default plot formats.

@jenshnielsen
Copy link
Member Author

Should we consider having travis build the full html docs including the pdf version? This is now pdf + 100 dpi png and not 80 dpf + 200 dpi png + pdf? It would be nice to have that in the devdocs.

A local benchmark shows that this increases the build time by about 2 min.

real    6m39.623s
user    6m20.541s
sys 0m9.313s

vs

real    4m55.762s
user    4m43.091s
sys 0m6.432s

Alternatively we could special case on the master branch and only build the pdf versions of the figures when we build directly from master not from PRs but that seems more confusing than it's worth?

@tacaswell tacaswell modified the milestone: 2.0 (style change major release) Jul 25, 2016
@tacaswell tacaswell merged commit 18ffa3e into matplotlib:v2.x Jul 26, 2016
@jenshnielsen jenshnielsen deleted the docsdpi branch July 26, 2016 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants