Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Reduce the use of XObjects in pdf backend [backport to 1.4.x] #3485

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.

Sign up for GitHub

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

Merged
merged 7 commits into from
Oct 19, 2014
Merged

Reduce the use of XObjects in pdf backend [backport to 1.4.x] #3485

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
07a58af
Count the uses of paths in a collection
jkseppan Sep 7, 2014
3d38633
Use the uses_per_path data in pdf backend
jkseppan Sep 7, 2014
353361c
Add the new test in the list of tests
jkseppan Sep 7, 2014
1ea88b0
Add new baseline images for two tests
jkseppan Sep 7, 2014
3a015fa
Handle path collections without stroke in PDF backend
mdboom Sep 8, 2014
878b681
Merge pull request #3 from mdboom/pdf-xobject-linewidth
jkseppan Sep 8, 2014
ef2c9fb
Reduce the emitting of object definitions in ps and svg
jkseppan Sep 21, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Reduce the emitting of object definitions in ps and svg 
The cost calculations are very rough, back-of-the-envelope.
In the example from #3345 I get significant file size reductions:

	2.3M	before.svg
	1.8M	after.svg
	916K	before.ps
	672K	after.ps
  • Loading branch information
@jkseppan
jkseppan committed Sep 21, 2014
commit ef2c9fbb871a528e75c52586f5db03a431debfa5
17 changes: 17 additions & 0 deletions lib/matplotlib/backends/backend_ps.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -637,6 +637,23 @@ def draw_path_collection(self, gc, master_transform, paths, all_transforms,
offsets, offsetTrans, facecolors, edgecolors,
linewidths, linestyles, antialiaseds, urls,
offset_position):
# Is the optimization worth it? Rough calculation:
# cost of emitting a path in-line is
# (len_path + 2) * uses_per_path
# cost of definition+use is
# (len_path + 3) + 3 * uses_per_path
len_path = len(paths[0].vertices) if len(paths) > 0 else 0
uses_per_path = self._iter_collection_uses_per_path(
paths, all_transforms, offsets, facecolors, edgecolors)
should_do_optimization = \
len_path + 3 * uses_per_path + 3 < (len_path + 2) * uses_per_path
if not should_do_optimization:
return RendererBase.draw_path_collection(
self, gc, master_transform, paths, all_transforms,
offsets, offsetTrans, facecolors, edgecolors,
linewidths, linestyles, antialiaseds, urls,
offset_position)

write = self._pswriter.write

path_codes = []
Expand Down
17 changes: 17 additions & 0 deletions lib/matplotlib/backends/backend_svg.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -607,6 +607,23 @@ def draw_path_collection(self, gc, master_transform, paths, all_transforms,
offsets, offsetTrans, facecolors, edgecolors,
linewidths, linestyles, antialiaseds, urls,
offset_position):
# Is the optimization worth it? Rough calculation:
# cost of emitting a path in-line is
# (len_path + 5) * uses_per_path
# cost of definition+use is
# (len_path + 3) + 9 * uses_per_path
len_path = len(paths[0].vertices) if len(paths) > 0 else 0
uses_per_path = self._iter_collection_uses_per_path(
paths, all_transforms, offsets, facecolors, edgecolors)
should_do_optimization = \
len_path + 9 * uses_per_path + 3 < (len_path + 5) * uses_per_path
if not should_do_optimization:
return RendererBase.draw_path_collection(
self, gc, master_transform, paths, all_transforms,
offsets, offsetTrans, facecolors, edgecolors,
linewidths, linestyles, antialiaseds, urls,
offset_position)

writer = self.writer
path_codes = []
writer.start('defs')
Expand Down