NotificationsYou must be signed in to change notification settings
Fork7.9k
Star21.4k

Docs change default dpi#6823

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

tacaswell merged 3 commits intomatplotlib:v2.xfromjenshnielsen:docsdpi

Jul 26, 2016

Merged

Docs change default dpi#6823

tacaswell merged 3 commits intomatplotlib:v2.xfromjenshnielsen:docsdpi

Jul 26, 2016

Conversation

Copy link

Member

jenshnielsen commentedJul 24, 2016

Closes#6636

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

jenshnielsen added2 commits

July 24, 2016 11:30

Change default dpi of small png images in docs

6bfe9e1

Closesmatplotlib#6636

Remove hires png from default docs html output

b710f54

jenshnielsen added the status: needs review label

Jul 24, 2016

Copy link

MemberAuthor

jenshnielsen commentedJul 24, 2016

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.623suser    6m20.541ssys 0m9.313s

real    4m55.762suser    4m43.091ssys 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?