MNT: Deprecate reimported functions in top-level namespace

timhoffm · timhoffm · commit 3483acf4d23c · 2024-08-15T15:15:21.000+02:00
diff --git a/doc/api/next_api_changes/deprecations/28728-TH.rst b/doc/api/next_api_changes/deprecations/28728-TH.rst
@@ -0,0 +1,10 @@
+matplotlib.validate_backend
+~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+...is deprecated. Please use `matplotlib.rcsetup.validate_backend` instead.
+
+
+matplotlib.sanitize_sequence
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+...is deprecated. Please use `matplotlib.cbook.sanitize_sequence` instead.
diff --git a/lib/matplotlib/__init__.py b/lib/matplotlib/__init__.py
@@ -157,10 +157,8 @@
 # cbook must import matplotlib only within function
 # definitions, so it is safe to import from it here.
 from . import _api, _version, cbook, _docstring, rcsetup
-from matplotlib.cbook import sanitize_sequence
 from matplotlib._api import MatplotlibDeprecationWarning
 from matplotlib.rcsetup import cycler  # noqa: F401
-from matplotlib.rcsetup import validate_backend
 
 
 _log = logging.getLogger(__name__)
@@ -1236,7 +1234,7 @@ def use(backend, *, force=True):
     matplotlib.pyplot.switch_backend
 
     """
-    name = validate_backend(backend)
+    name = rcsetup.validate_backend(backend)
     # don't (prematurely) resolve the "auto" backend setting
     if rcParams._get_backend_or_none() == name:
         # Nothing to do if the requested backend is already set
@@ -1510,6 +1508,16 @@ def inner(ax, *args, data=None, **kwargs):
 _log.debug('platform is %s', sys.platform)
 
 
+@_api.deprecated("3.10", alternative="matplotlib.cbook.sanitize_sequence")
+def sanitize_sequence(data):
+    return cbook.sanitize_sequence(data)
+
+
+@_api.deprecated("3.10", alternative="matplotlib.rcsetup.validate_backend")
+def validate_backend(s):
+    return rcsetup.validate_backend(s)
+
+
 # workaround: we must defer colormaps import to after loading rcParams, because
 # colormap creation depends on rcParams
 from matplotlib.cm import _colormaps as colormaps  # noqa: E402
