Merge pull request matplotlib#31992 from HakonSohoel/fix-inverted-3d-axis-placement

Fix: Axis and grid panes out of sync when inverting an axis

Author: QuLogic

lib/mpl_toolkits/mplot3d/axis3d.py

@@ -269,8 +269,13 @@ def get_rotate_label(self, text):
     def _get_coord_info(self):
         # Get scaled limits directly from the axes helper
         xmin, xmax, ymin, ymax, zmin, zmax = self.axes._get_scaled_limits()
-        mins = np.array([xmin, ymin, zmin])
-        maxs = np.array([xmax, ymax, zmax])
+        # When axes are inverted the min and max values returned by _get_scaled_limits
+        # are reversed (min > max). Here we need to ensure max > min so that the axis
+        # lines and grid panes are placed consistently regardless of inversion.
+        # Note that we cannot ensure this inside _get_scaled_limits itself, as other
+        # callers rely on the signed range to detect and handle axis inversion.
+        mins = np.array([min(xmin, xmax), min(ymin, ymax), min(zmin, zmax)])
+        maxs = np.array([max(xmin, xmax), max(ymin, ymax), max(zmin, zmax)])
 
         # Get data-space bounds for _transformed_cube
         bounds = (*self.axes.get_xbound(),

lib/mpl_toolkits/mplot3d/tests/baseline_images/test_axes3d/axis_positions_inverted.png

@@ -72,6 +72,31 @@ def test_axis_positions():
         ax.set(xlabel='x', ylabel='y', zlabel='z', title=title)
 
 
+@mpl3d_image_comparison(['axis_positions_inverted.png'], remove_text=False,
+                        style='mpl20')
+def test_axis_positions_inverted():
+    # Regression test for https://github.com/matplotlib/matplotlib/issues/31989
+    # Check the visual placement of axes, ticks and labels for
+    # every combination of inverted x, y and z axes.
+    combinations = list(itertools.product([False, True], repeat=3))
+    fig, axs = plt.subplots(2, 4, figsize=(10, 6),
+                            subplot_kw={'projection': '3d'})
+    for ax, (invert_x, invert_y, invert_z) in zip(axs.flatten(), combinations):
+        # Plot an asymmetric line so that inverting an axis visibly mirrors the data,
+        # ensuring the projection is exercised for every inversion combination
+        ax.plot([0, 1, 1], [0, 0, 1], [0, 1, 1])
+        if invert_x:
+            ax.invert_xaxis()
+        if invert_y:
+            ax.invert_yaxis()
+        if invert_z:
+            ax.invert_zaxis()
+        title = (f'{"-" if invert_x else ""}x, '
+                 f'{"-" if invert_y else ""}y, '
+                 f'{"-" if invert_z else ""}z')
+        ax.set(xlabel='x', ylabel='y', zlabel='z', title=title)
+
+
 @mpl3d_image_comparison(['aspects.png'], remove_text=False, style='mpl20')
 def test_aspects():
     aspects = ('auto', 'equal', 'equalxy', 'equalyz', 'equalxz', 'equal')