CI/BLD: Bump min python to 3.10, since 3.9 is at EOL (#643)

Author: Jacob-Stevens-Haas

.github/workflows/main.yml

@@ -19,7 +19,7 @@ jobs:
       - name: Set up Python
         uses: actions/setup-python@v4
         with:
-          python-version: "3.9"
+          python-version: "3.10"
       - name: Linting
         run: |
           pip install pre-commit
@@ -34,7 +34,7 @@ jobs:
     - name: Set up Python
       uses: actions/setup-python@v4
       with:
-        python-version: "3.9"
+        python-version: "3.10"
     - name: build
       run: pip install .
     - name: test import
@@ -71,7 +71,7 @@ jobs:
       fail-fast: false
       max-parallel: 4
       matrix:
-        python-version: ["3.9", "3.11"]
+        python-version: ["3.10", "3.12"]
 
     steps:
     - uses: actions/checkout@v4

.github/workflows/notebooks.yml

@@ -30,7 +30,7 @@ jobs:
       - name: Set up Python
         uses: actions/setup-python@v3
         with:
-          python-version: "3.9"
+          python-version: "3.10"
       - name: Install dependencies
         run: |
           pip install .[cvxpy,miosr] sympy nbconvert jupyter matplotlib seaborn pandas dysts

.github/workflows/release.yml

@@ -14,7 +14,7 @@ jobs:
       - name: Set up Python
         uses: actions/setup-python@v3
         with:
-          python-version: "3.9"
+          python-version: "3.10"
       - name: Install Build
         run: |
           python -m pip install --upgrade pip

pyproject.toml

@@ -12,13 +12,14 @@ authors = [
     {name="Alan Kaptanoglu", email="akaptano@uw.edu"}
 ]
 license = {text = "MIT"}
-requires-python = ">=3.8"
+requires-python = ">=3.10"
 dynamic = ["version"]
 classifiers = [
     "Programming Language :: Python",
-    "Programming Language :: Python :: 3.8",
-    "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13",
     "Development Status :: 5 - Production/Stable",
     "Intended Audience :: Science/Research",
     "License :: OSI Approved :: MIT License",

pysindy/pysindy.py

@@ -1,4 +1,3 @@
-import sys
 import warnings
 from abc import ABC
 from abc import abstractmethod
@@ -814,32 +813,14 @@ def _check_multiple_trajectories(x, x_dot, u) -> bool:
 
     """
     SequenceOrNone = Union[Sequence, None]
-    if sys.version_info.minor < 10:
-        mixed_trajectories = (
-            isinstance(x, Sequence)
-            and (
-                not isinstance(x_dot, Sequence)
-                and x_dot is not None
-                or not isinstance(u, Sequence)
-                and u is not None
-            )
-            or isinstance(x_dot, Sequence)
-            and not isinstance(x, Sequence)
-            or isinstance(u, Sequence)
-            and not isinstance(x, Sequence)
-        )
-    else:
-        mixed_trajectories = (
-            isinstance(x, Sequence)
-            and (
-                not isinstance(x_dot, SequenceOrNone)
-                or not isinstance(u, SequenceOrNone)
-            )
-            or isinstance(x_dot, Sequence)
-            and not isinstance(x, Sequence)
-            or isinstance(u, Sequence)
-            and not isinstance(x, Sequence)
-        )
+    mixed_trajectories = (
+        isinstance(x, Sequence)
+        and (not isinstance(x_dot, SequenceOrNone) or not isinstance(u, SequenceOrNone))
+        or isinstance(x_dot, Sequence)
+        and not isinstance(x, Sequence)
+        or isinstance(u, Sequence)
+        and not isinstance(x, Sequence)
+    )
     if mixed_trajectories:
         raise TypeError(
             "If x, x_dot, or u are a Sequence of trajectories, each must be a Sequence"