Merge pull request #25681 from stefmolin/null-bbox

ksunden · web-flow · commit 0e7f29e4f557 · 2023-04-17T00:48:36.000-05:00
BUG: Return null Bbox when there is no intersection for bar_label center.
diff --git a/lib/matplotlib/axes/_axes.py b/lib/matplotlib/axes/_axes.py
@@ -2769,7 +2769,7 @@ def sign(x):
                     lambda r, b=bar:
                         mtransforms.Bbox.intersection(
                             b.get_window_extent(r), b.get_clip_box()
-                        )
+                        ) or mtransforms.Bbox.null()
                 )
             else:  # edge
                 if orientation == "vertical":
diff --git a/lib/matplotlib/tests/test_axes.py b/lib/matplotlib/tests/test_axes.py
@@ -8040,6 +8040,19 @@ def test_centered_bar_label_nonlinear():
     ax.set_axis_off()
 
 
+def test_centered_bar_label_label_beyond_limits():
+    fig, ax = plt.subplots()
+
+    last = 0
+    for label, value in zip(['a', 'b', 'c'], [10, 20, 50]):
+        bar_container = ax.barh('col', value, label=label, left=last)
+        ax.bar_label(bar_container, label_type='center')
+        last += value
+    ax.set_xlim(None, 20)
+
+    fig.draw_without_rendering()
+
+
 def test_bar_label_location_errorbars():
     ax = plt.gca()
     xs, heights = [1, 2], [3, -4]

Original file line number	Diff line number	Diff line change
`@@ -2769,7 +2769,7 @@ def sign(x):`
`2769`	`2769`	`lambda r, b=bar:`
`2770`	`2770`	`mtransforms.Bbox.intersection(`
`2771`	`2771`	`b.get_window_extent(r), b.get_clip_box()`
`2772`		`- )`
	`2772`	`+ ) or mtransforms.Bbox.null()`
`2773`	`2773`	`)`
`2774`	`2774`	`else: # edge`
`2775`	`2775`	`if orientation == "vertical":`