Remaining mpl3.6 deprecations

ksunden · ksunden · commit ec3c8beb8c2e · 2023-04-17T14:42:47.000-05:00
pending deprecations remain in axis3D and in Figure
diff --git a/ci/mypy-stubtest-allowlist.txt b/ci/mypy-stubtest-allowlist.txt
@@ -84,13 +84,10 @@ matplotlib.transforms.TransformWrapper.input_dims
 matplotlib.transforms.TransformWrapper.is_separable
 matplotlib.transforms.TransformWrapper.output_dims
 
-# 3.6 deprecations
-matplotlib.colorbar.Colorbar.__init__
-matplotlib.figure.Figure.callbacks
+# 3.6 Pending deprecations
 matplotlib.figure.Figure.set_constrained_layout
 matplotlib.figure.Figure.set_constrained_layout_pads
 matplotlib.figure.Figure.set_tight_layout
-matplotlib.figure.SubFigure.callbacks
 
 # 3.7 deprecations
 matplotlib.cm.register_cmap
@@ -103,7 +100,6 @@ matplotlib.widgets.Slider.__init__
 matplotlib.widgets.RangeSlider.__init__
 matplotlib.widgets.TextBox.__init__
 matplotlib.widgets.Cursor.__init__
-matplotlib.widgets.MultiCursor.__init__
 matplotlib.widgets.SpanSelector.__init__
 matplotlib.widgets.ToolLineHandles.__init__
 matplotlib.widgets.ToolHandles.__init__
diff --git a/lib/matplotlib/colorbar.py b/lib/matplotlib/colorbar.py
@@ -278,7 +278,6 @@ class Colorbar:
 
     n_rasterize = 50  # rasterize solids if number of colors >= n_rasterize
 
-    @_api.delete_parameter("3.6", "filled")
     def __init__(self, ax, mappable=None, *, cmap=None,
                  norm=None,
                  alpha=None,
@@ -291,7 +290,6 @@ def __init__(self, ax, mappable=None, *, cmap=None,
                  ticks=None,
                  format=None,
                  drawedges=False,
-                 filled=True,
                  extendfrac=None,
                  extendrect=False,
                  label='',
@@ -305,6 +303,7 @@ def __init__(self, ax, mappable=None, *, cmap=None,
         cmap = mappable.cmap
         norm = mappable.norm
 
+        filled = True
         if isinstance(mappable, contour.ContourSet):
             cs = mappable
             alpha = cs.get_alpha()
diff --git a/lib/matplotlib/colorbar.pyi b/lib/matplotlib/colorbar.pyi
@@ -57,7 +57,6 @@ class Colorbar:
         ticks: Sequence[float] | Locator | None = ...,
         format: str | Formatter | None = ...,
         drawedges: bool = ...,
-        filled: bool = ...,
         extendfrac: Literal["auto"] | float | Sequence[float] | None = ...,
         extendrect: bool = ...,
         label: str = ...,
diff --git a/lib/matplotlib/figure.py b/lib/matplotlib/figure.py
@@ -2142,10 +2142,6 @@ class SubFigure(FigureBase):
 
     See :doc:`/gallery/subplots_axes_and_figures/subfigures`
     """
-    callbacks = _api.deprecated(
-            "3.6", alternative=("the 'resize_event' signal in "
-                                "Figure.canvas.callbacks")
-            )(property(lambda self: self._fig_callbacks))
 
     def __init__(self, parent, subplotspec, *,
                  facecolor=None,
@@ -2194,7 +2190,6 @@ def __init__(self, parent, subplotspec, *,
         self._subplotspec = subplotspec
         self._parent = parent
         self.figure = parent.figure
-        self._fig_callbacks = parent._fig_callbacks
 
         # subfigures use the parent axstack
         self._axstack = parent._axstack
@@ -2355,12 +2350,6 @@ class Figure(FigureBase):
         depending on the renderer option_image_nocomposite function.  If
         *suppressComposite* is a boolean, this will override the renderer.
     """
-    # Remove the self._fig_callbacks properties on figure and subfigure
-    # after the deprecation expires.
-    callbacks = _api.deprecated(
-        "3.6", alternative=("the 'resize_event' signal in "
-                            "Figure.canvas.callbacks")
-        )(property(lambda self: self._fig_callbacks))
 
     def __str__(self):
         return "Figure(%gx%g)" % tuple(self.bbox.size)
@@ -2501,7 +2490,6 @@ def __init__(self,
             # everything is None, so use default:
             self.set_layout_engine(layout=layout)
 
-        self._fig_callbacks = cbook.CallbackRegistry(signals=["dpi_changed"])
         # Callbacks traditionally associated with the canvas (and exposed with
         # a proxy property), but that actually need to be on the figure for
         # pickling.
@@ -2750,7 +2738,6 @@ def _set_dpi(self, dpi, forward=True):
         self.dpi_scale_trans.clear().scale(dpi)
         w, h = self.get_size_inches()
         self.set_size_inches(w, h, forward=forward)
-        self._fig_callbacks.process('dpi_changed', self)
 
     dpi = property(_get_dpi, _set_dpi, doc="The resolution in dots per inch.")
 
diff --git a/lib/matplotlib/tri/_tripcolor.py b/lib/matplotlib/tri/_tripcolor.py
@@ -80,10 +80,7 @@ def tripcolor(ax, *args, alpha=1.0, norm=None, cmap=None, vmin=None,
                 "tripcolor() missing 1 required positional argument: 'c'; or "
                 "1 required keyword-only argument: 'facecolors'")
         elif len(args) > 1:
-            _api.warn_deprecated(
-                "3.6", message=f"Additional positional parameters "
-                f"{args[1:]!r} are ignored; support for them is deprecated "
-                f"since %(since)s and will be removed %(removal)s")
+            raise TypeError(f"Unexpected positional parameters: {args[1:]!r}")
         c = np.asarray(args[0])
         if len(c) == len(tri.x):
             # having this before the len(tri.triangles) comparison gives
diff --git a/lib/matplotlib/widgets.py b/lib/matplotlib/widgets.py
@@ -2429,13 +2429,9 @@ def get_visible(self):
 
     @property
     def visible(self):
+        _api.warn_deprecated("3.6", alternative="get_visible")
         return self.get_visible()
 
-    @visible.setter
-    def visible(self, visible):
-        _api.warn_deprecated("3.6", alternative="set_visible")
-        self.set_visible(visible)
-
     def clear(self):
         """Clear the selection and set the selector ready to make a new one."""
         self._clear_without_update()
diff --git a/lib/matplotlib/widgets.pyi b/lib/matplotlib/widgets.pyi
@@ -259,6 +259,7 @@ class MultiCursor(Widget):
         self,
         canvas: Any,
         axes: Sequence[Axes],
+        *,
         useblit: bool = ...,
         horizOn: bool = ...,
         vertOn: bool = ...,
@@ -297,8 +298,6 @@ class _SelectorWidget(AxesWidget):
     def get_visible(self) -> bool: ...
     @property
     def visible(self) -> bool: ...
-    @visible.setter
-    def visible(self, visible: bool) -> None: ...
     def clear(self) -> None: ...
     @property
     def artists(self) -> tuple[Artist]: ...
