Merge pull request #12668 from meeseeksmachine/auto-backport-of-pr-12651-on-v3.0.x

jklymak · web-flow · commit b47faf4245b6 · 2018-10-29T13:51:21.000-07:00
Backport PR #12651 on branch v3.0.x (FIX: ignore non-finite bbox)
diff --git a/lib/matplotlib/axes/_base.py b/lib/matplotlib/axes/_base.py
@@ -4238,7 +4238,9 @@ def get_tightbbox(self, renderer, call_axes_locator=True,
 
         for a in bbox_artists:
             bbox = a.get_tightbbox(renderer)
-            if bbox is not None and (bbox.width != 0 or bbox.height != 0):
+            if (bbox is not None and
+                    (bbox.width != 0 or bbox.height != 0) and
+                    np.isfinite(bbox.width) and np.isfinite(bbox.height)):
                 bb.append(bbox)
 
         _bbox = mtransforms.Bbox.union(
diff --git a/lib/matplotlib/tests/test_axes.py b/lib/matplotlib/tests/test_axes.py
@@ -5841,3 +5841,10 @@ class DummySubplot(matplotlib.axes.SubplotBase, Dummy):
     FactoryDummySubplot = matplotlib.axes.subplot_class_factory(Dummy)
 
     assert DummySubplot is FactoryDummySubplot
+
+
+def test_gettightbbox_ignoreNaN():
+    fig, ax = plt.subplots()
+    t = ax.text(np.NaN, 1, 'Boo')
+    renderer = fig.canvas.get_renderer()
+    np.testing.assert_allclose(ax.get_tightbbox(renderer).width, 532.444444)
