DEP: Support Python 3.13

Author: larsoner

.github/workflows/tests.yml

@@ -56,6 +56,9 @@ jobs:
       fail-fast: false
       matrix:
         include:
+          - os: ubuntu-latest
+            python: '3.13'
+            kind: pip
           - os: ubuntu-latest
             python: '3.12'
             kind: pip-pre
@@ -92,13 +95,6 @@ jobs:
         with:
           python-version: ${{ matrix.python }}
         if: startswith(matrix.kind, 'pip')
-      # Python (if conda)
-      - name: Remove numba and dipy
-        run: |  # TODO: Remove when numba 0.59 and dipy 1.8 land on conda-forge
-          sed -i '/numba/d' environment.yml
-          sed -i '/dipy/d' environment.yml
-          sed -i 's/- mne$/- mne-base/' environment.yml
-        if: matrix.os == 'ubuntu-latest' && startswith(matrix.kind, 'conda') && matrix.python == '3.12'
       - uses: mamba-org/setup-micromamba@v2
         with:
           environment-file: ${{ env.CONDA_ENV }}

doc/changes/devel/dependency.rst

@@ -0,0 +1 @@
+Compatibility improved for Python 3.13, by `Eric Larson`.

mne/utils/docs.py

@@ -5109,6 +5109,8 @@ def copy_doc(source):
     This is useful when inheriting from a class and overloading a method. This
     decorator can be used to copy the docstring of the original method.
 
+    Docstrings are processed by :func:`python:inspect.cleandoc` before being used.
+
     Parameters
     ----------
     source : function
@@ -5131,15 +5133,16 @@ def copy_doc(source):
     ...         ''' this gets appended'''
     ...         pass
     >>> print(B.m1.__doc__)
-    Docstring for m1 this gets appended
+    Docstring for m1
+    this gets appended
     """
 
     def wrapper(func):
         if source.__doc__ is None or len(source.__doc__) == 0:
             raise ValueError("Cannot copy docstring: docstring was empty.")
         doc = source.__doc__
         if func.__doc__ is not None:
-            doc += func.__doc__
+            doc += f"\n{inspect.cleandoc(func.__doc__)}"
         func.__doc__ = doc
         return func
 
@@ -5158,6 +5161,10 @@ def copy_function_doc_to_method_doc(source):
     function.  This pattern is prevalent in for example the plotting functions
     of MNE.
 
+    Docstrings are parsed by :func:`python:inspect.cleandoc` before being used.
+    If indentation and newlines are important, make the first line ``.``, and the dot
+    will be removed and all following lines dedented jointly.
+
     Parameters
     ----------
     source : function
@@ -5193,7 +5200,8 @@ def copy_function_doc_to_method_doc(source):
     >>> class A:
     ...     @copy_function_doc_to_method_doc(plot_function)
     ...     def plot(self, a, b):
-    ...         '''
+    ...         '''.
+    ...
     ...         Notes
     ...         -----
     ...         .. versionadded:: 0.13.0
@@ -5202,26 +5210,31 @@ def copy_function_doc_to_method_doc(source):
     >>> print(A.plot.__doc__)
     Docstring for plotting function.
     <BLANKLINE>
-        Parameters
-        ----------
-        a : int
-            Some parameter
-        b : int
-            Some parameter
-    <BLANKLINE>
-            Notes
-            -----
-            .. versionadded:: 0.13.0
+    Parameters
+    ----------
+    a : int
+        Some parameter
+    b : int
+        Some parameter
     <BLANKLINE>
+    Notes
+    -----
+    .. versionadded:: 0.13.0
     """  # noqa: D410, D411, D214, D215
 
     def wrapper(func):
-        doc = source.__doc__.split("\n")
+        # Work with cleandoc'ed sources (py3.13-compat)
+        doc = inspect.cleandoc(source.__doc__).split("\n")
+        if func.__doc__ is not None:
+            func_doc = inspect.cleandoc(func.__doc__)
+            if func_doc[:2] == ".\n":
+                func_doc = func_doc[2:]
+            func_doc = f"\n{func_doc}"
+        else:
+            func_doc = ""
+
         if len(doc) == 1:
-            doc = doc[0]
-            if func.__doc__ is not None:
-                doc += func.__doc__
-            func.__doc__ = doc
+            func.__doc__ = f"{doc[0]}{func_doc}"
             return func
 
         # Find parameter block
@@ -5269,7 +5282,7 @@ def wrapper(func):
                 break
         else:
             # End of docstring reached
-            first_parameter_end = line
+            first_parameter_end = line + 1
             first_parameter = parameter_block
 
         # Copy the docstring, but remove the first parameter
@@ -5278,9 +5291,7 @@ def wrapper(func):
             + "\n"
             + "\n".join(doc[first_parameter_end:])
         )
-        if func.__doc__ is not None:
-            doc += func.__doc__
-        func.__doc__ = doc
+        func.__doc__ = f"{doc}{func_doc}"
         return func
 
     return wrapper

mne/utils/tests/test_config.py

@@ -129,7 +129,7 @@ def test_sys_info_complete():
     pyproject = tomllib.loads(pyproject.read_text("utf-8"))
     deps = pyproject["project"]["optional-dependencies"]["test_extra"]
     for dep in deps:
-        dep = dep.split("[")[0].split(">")[0]
+        dep = dep.split("[")[0].split(">")[0].strip()
         assert f" {dep}" in out, f"Missing in dev config: {dep}"
 
 

mne/utils/tests/test_docs.py

@@ -195,28 +195,29 @@ def method_f3(self):
 
     assert (
         A.method_f1.__doc__
-        == """Docstring for f1.
-
-        Parameters
-        ----------
-        a : int
-            Parameter a
-        b : int
-            Parameter b
-        """
+        == """\
+Docstring for f1.
+
+Parameters
+----------
+a : int
+    Parameter a
+b : int
+    Parameter b"""
     )
 
     assert (
         A.method_f2.__doc__
-        == """Docstring for f2.
+        == """\
+Docstring for f2.
 
-        Returns
-        -------
-        nothing.
-        method_f3 own docstring"""
+Returns
+-------
+nothing.
+method_f3 own docstring"""
     )
 
-    assert A.method_f3.__doc__ == "Docstring for f3.\n\n        "
+    assert A.method_f3.__doc__ == "Docstring for f3.\n\n"
     pytest.raises(ValueError, copy_function_doc_to_method_doc(f5), A.method_f1)
 
 

tools/github_actions_dependencies.sh

@@ -20,6 +20,11 @@ if [ ! -z "$CONDA_ENV" ]; then
 		INSTALL_KIND="test"
 		STD_ARGS="--progress-bar off"
 	fi
+elif [[ "${MNE_CI_KIND}" == "pip" ]]; then
+	# Only used for 3.13 at the moment, just get test deps plus a few extras
+	# that we know are available
+	INSTALL_ARGS="nibabel scikit-learn numpydoc PySide6 mne-qt-browser"
+	INSTALL_KIND="test"
 else
 	test "${MNE_CI_KIND}" == "pip-pre"
 	STD_ARGS="$STD_ARGS --pre"