Shorter svg files #5651

Merged
merged 3 commits into from Dec 11, 2015

Conversation

Projects
None yet
2 participants
Owner

mdboom commented Dec 10, 2015

As mentioned in this comment the change to use %f over str in the SVG backend may result in larger file sizes. This addresses that, while still keeping Python 2/3 compatibility. There was only one test image affected by this change.

mdboom added some commits Dec 10, 2015

@mdboom mdboom Reduce SVG file size d2d4c80
@mdboom mdboom Update test
2df702d

mdboom added the needs_review label Dec 10, 2015

mdboom added this to the next major release (2.0) milestone Dec 10, 2015

@mdboom mdboom Update test image
7163922
Owner

mdboom commented Dec 11, 2015

I think this good to go.

@efiring efiring added a commit that referenced this pull request Dec 11, 2015

@efiring efiring Merge pull request #5651 from mdboom/shorter-svg-files
Shorter svg files
c95e64d

@efiring efiring merged commit c95e64d into matplotlib:master Dec 11, 2015

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.001%) to 68.251%
Details

efiring removed the needs_review label Dec 11, 2015

Owner

efiring commented Dec 11, 2015

Nice!

QuLogic referenced this pull request Dec 11, 2015

Merged

Backport #5307 to v2.0.x #5652

@efiring efiring added a commit that referenced this pull request Dec 11, 2015

@efiring @mdboom efiring + mdboom Merge pull request #5651 from mdboom/shorter-svg-files
Shorter svg files
92e57c8
Owner

mdboom commented Dec 11, 2015

Backported to 2.0.x as 92e57c8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment