Merge pull request #27647 from saranti/minorticks

dstansby · dstansby · commit 697c7cb6dfac · 2024-01-17T12:00:39.000Z
Fix error that occurs when minorticks are on multi-Axes Figure with more than one boxplot (cherry picked from commit b03407b)
diff --git a/lib/matplotlib/tests/test_ticker.py b/lib/matplotlib/tests/test_ticker.py
@@ -1797,3 +1797,24 @@ def test_set_offset_string(formatter):
     assert formatter.get_offset() == ''
     formatter.set_offset_string('mpl')
     assert formatter.get_offset() == 'mpl'
+
+
+def test_minorticks_on_multi_fig():
+    """
+    Turning on minor gridlines in a multi-Axes Figure
+    that contains more than one boxplot and shares the x-axis
+    should not raise an exception.
+    """
+    fig, ax = plt.subplots()
+
+    ax.boxplot(np.arange(10), positions=[0])
+    ax.boxplot(np.arange(10), positions=[0])
+    ax.boxplot(np.arange(10), positions=[1])
+
+    ax.grid(which="major")
+    ax.grid(which="minor")
+    ax.minorticks_on()
+    fig.draw_without_rendering()
+
+    assert ax.get_xgridlines()
+    assert isinstance(ax.xaxis.get_minor_locator(), mpl.ticker.AutoMinorLocator)
diff --git a/lib/matplotlib/ticker.py b/lib/matplotlib/ticker.py
@@ -2895,7 +2895,7 @@ def __call__(self):
                                'logarithmic scale')
             return []
 
-        majorlocs = self.axis.get_majorticklocs()
+        majorlocs = np.unique(self.axis.get_majorticklocs())
         try:
             majorstep = majorlocs[1] - majorlocs[0]
         except IndexError:
