Transform ParasiteAxesBase._update_viewlim into standard callback.

anntzer · anntzer · commit 1f4c1443fe4b · 2022-11-21T17:54:14.000+01:00
update_viewlim as an independent method was deprecated in da2f77a and has since then been removed. We could just inline _update_viewlim into apply_aspect, but transforming it into a callback on xlim_changed/ylim_changed helps moving towards the removal of ParasiteAxes as a separate class and its replacement by standard (child) axes with some extra attributes/callbacks set.
diff --git a/doc/api/prev_api_changes/api_changes_3.6.0/removals.rst b/doc/api/prev_api_changes/api_changes_3.6.0/removals.rst
@@ -152,7 +152,7 @@ The following class methods have been removed:
   ``RendererAgg.tostring_rgba_minimized()``
 - ``NavigationToolbar2Wx.get_canvas()``
 
-- ``ParasiteAxesBase.update_viewlim()``; use `.ParasiteAxesBase.apply_aspect`
+- ``ParasiteAxesBase.update_viewlim()``; use ``ParasiteAxesBase.apply_aspect``
   instead.
 - ``Subplot.get_geometry()``; use ``SubplotBase.get_subplotspec`` instead.
 - ``Subplot.change_geometry()``; use ``SubplotBase.set_subplotspec`` instead.
diff --git a/lib/mpl_toolkits/axes_grid1/parasite_axes.py b/lib/mpl_toolkits/axes_grid1/parasite_axes.py
@@ -14,6 +14,7 @@ def __init__(self, parent_axes, aux_transform=None,
         self.set_viewlim_mode(viewlim_mode)
         kwargs["frameon"] = False
         super().__init__(parent_axes.figure, parent_axes._position, **kwargs)
+        parent_axes.callbacks.connect("xlim_changed", self._sync_lims)
 
     def clear(self):
         super().clear()
@@ -52,23 +53,18 @@ def set_viewlim_mode(self, mode):
     def get_viewlim_mode(self):
         return self._viewlim_mode
 
-    def _update_viewlim(self):  # Inline after deprecation elapses.
-        viewlim = self._parent_axes.viewLim.frozen()
+    def _sync_lims(self, parent):
+        viewlim = parent.viewLim.frozen()
         mode = self.get_viewlim_mode()
         if mode is None:
             pass
         elif mode == "equal":
-            self.axes.viewLim.set(viewlim)
+            self.viewLim.set(viewlim)
         elif mode == "transform":
-            self.axes.viewLim.set(
-                viewlim.transformed(self.transAux.inverted()))
+            self.viewLim.set(viewlim.transformed(self.transAux.inverted()))
         else:
             _api.check_in_list([None, "equal", "transform"], mode=mode)
 
-    def apply_aspect(self, position=None):
-        self._update_viewlim()
-        super().apply_aspect()
-
     # end of aux_transform support
 
 
