Update lib/matplotlib/ticker.py

aalmasmari · oscargus · alialmas · commit 229d11626965 · 2022-12-11T22:02:39.000-05:00
Co-authored-by: Oscar Gustafsson &lt;oscar.gustafsson@gmail.com&gt;
diff --git a/lib/matplotlib/tests/test_ticker.py b/lib/matplotlib/tests/test_ticker.py
@@ -106,6 +106,26 @@ def test_basic(self):
         (1, 0)   # a single major tick => no minor tick
     ]
 
+    def test_first_and_last_minorticks(self):
+        """
+        Test that first and last minor tick appear as expected.
+        """
+        # This test is related to issue #22331
+        fig = plt.figure()
+        ax = plt.gca()
+        ax.set_xlim(-1.9, 1.9)
+        ax.xaxis.set_minor_locator(mticker.AutoMinorLocator())
+        test_value = np.array([-1.9, -1.8, -1.7, -1.6, -1.4, -1.3, -1.2, -1.1,
+                               -0.9, -0.8, -0.7, -0.6, -0.4, -0.3, -0.2, -0.1,
+                               0.1, 0.2, 0.3, 0.4, 0.6, 0.7, 0.8, 0.9, 1.1,
+                               1.2, 1.3, 1.4, 1.6, 1.7, 1.8, 1.9])
+        assert_almost_equal(ax.xaxis.get_ticklocs(minor=True), test_value)
+
+        ax.set_xlim(-5, 5)
+        test_value = np.array([-5.0, -4.5, -3.5, -3.0, -2.5, -1.5, -1.0, -0.5,
+                               0.5, 1.0, 1.5, 2.5, 3.0, 3.5, 4.5, 5.0])
+        assert_almost_equal(ax.xaxis.get_ticklocs(minor=True), test_value)
+
     @pytest.mark.parametrize('nb_majorticks, expected_nb_minorticks', params)
     def test_low_number_of_majorticks(
             self, nb_majorticks, expected_nb_minorticks):
diff --git a/lib/matplotlib/ticker.py b/lib/matplotlib/ticker.py
@@ -2953,10 +2953,9 @@ def __call__(self):
             vmin, vmax = vmax, vmin
 
         t0 = majorlocs[0]
-        tmin = round((vmin - t0) / minorstep) - 1
+        tmin = round((vmin - t0) / minorstep)
         tmax = round((vmax - t0) / minorstep) + 1
-        locs = np.arange((tmin + 1) * minorstep,
-                         tmax * minorstep, minorstep) + t0
+        locs = (np.arange(tmin, tmax) * minorstep) + t0
 
         return self.raise_if_exceeds(locs)
 
