Reduce number of marker objects in pdf output

jkseppan · jkseppan · commit 24cc7a7d2ba4 · 2009-02-02T19:35:43.000Z
svn path=/trunk/matplotlib/; revision=6868
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,3 +1,5 @@
+2009-02-02 Reduce number of marker XObjects in pdf output - JKS
+
 2009-02-02 Change default resolution on polar plot to 1 - MGD
 
 2009-02-02 Avoid malloc errors in ttconv for fonts that don't have
diff --git a/lib/matplotlib/backends/backend_pdf.py b/lib/matplotlib/backends/backend_pdf.py
@@ -1109,26 +1109,28 @@ def writeImages(self):
 
     def markerObject(self, path, trans, fillp, lw):
         """Return name of a marker XObject representing the given path."""
-        key = (path, trans, fillp is not None, lw)
+        pathops = self.pathOperations(path, trans)
+        key = (tuple(pathops), bool(fillp))
         result = self.markers.get(key)
         if result is None:
             name = Name('M%d' % len(self.markers))
             ob = self.reserveObject('marker %d' % len(self.markers))
-            self.markers[key] = (name, ob, path, trans, fillp, lw)
+            bbox = path.get_extents(trans)
+            self.markers[key] = [name, ob, bbox, lw]
         else:
+            if result[-1] < lw:
+                result[-1] = lw
             name = result[0]
         return name
 
     def writeMarkers(self):
-        for tup in self.markers.values():
-            name, object, path, trans, fillp, lw = tup
-            bbox = path.get_extents(trans)
+        for (pathops, fillp),(name, ob, bbox, lw) in self.markers.iteritems():
             bbox = bbox.padded(lw * 0.5)
             self.beginStream(
-                object.id, None,
+                ob.id, None,
                 {'Type': Name('XObject'), 'Subtype': Name('Form'),
                  'BBox': list(bbox.extents) })
-            self.writePath(path, trans)
+            self.output(*pathops)
             if fillp:
                 self.output(Op.fill_stroke)
             else:
@@ -1280,10 +1282,17 @@ def draw_image(self, x, y, im, bbox, clippath=None, clippath_trans=None):
 
     def draw_path(self, gc, path, transform, rgbFace=None):
         self.check_gc(gc, rgbFace)
-        stream = self.file.writePath(path, transform, rgbFace is None)
+        self.file.writePath(path, transform, rgbFace is None)
         self.file.output(self.gc.paint())
 
     def draw_markers(self, gc, marker_path, marker_trans, path, trans, rgbFace=None):
+        # For simple paths or small numbers of markers, don't bother
+        # making an XObject
+        if len(path) * len(marker_path) <= 10:
+            RendererBase.draw_markers(self, gc, marker_path, marker_trans,
+                                      path, trans, rgbFace)
+            return
+
         self.check_gc(gc, rgbFace)
         fillp = rgbFace is not None
 

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+2009-02-02 Reduce number of marker XObjects in pdf output - JKS`
	`2`	`+`
`1`	`3`	`2009-02-02 Change default resolution on polar plot to 1 - MGD`
`2`	`4`
`3`	`5`	`2009-02-02 Avoid malloc errors in ttconv for fonts that don't have`