Fix for issue #2872. In case of missing values (e.g., zeros when using a logarithmic scale), draw_path_collection may get offsets containing NaN's. In that case, using CGContextTranslateCTM once with translation and once with -translation will not restore the original CTM. This bugfix adds a check for NaN/inf.

Michiel de Hoon · Michiel de Hoon · commit 74273e8292c3 · 2014-03-07T19:08:16.000+09:00
diff --git a/src/_macosx.m b/src/_macosx.m
@@ -1619,6 +1619,8 @@ static int _transformation_converter(PyObject* object, void* pointer)
                 translation.x = - (origin.x - translation.x);
                 translation.y = - (origin.y - translation.y);
             }
+            /* in case of missing values, translation may contain NaN's */
+            if (!isfinite(translation.x) || !isfinite(translation.y)) continue;
             CGContextTranslateCTM(cr, translation.x, translation.y);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -1619,6 +1619,8 @@ static int _transformation_converter(PyObject* object, void* pointer)`
`1619`	`1619`	`translation.x = - (origin.x - translation.x);`
`1620`	`1620`	`translation.y = - (origin.y - translation.y);`
`1621`	`1621`	`}`
	`1622`	`+ /* in case of missing values, translation may contain NaN's */`
	`1623`	`+ if (!isfinite(translation.x) \|\| !isfinite(translation.y)) continue;`
`1622`	`1624`	`CGContextTranslateCTM(cr, translation.x, translation.y);`
`1623`	`1625`	`}`
`1624`	`1626`