dropped 3.9 support (#944)

* dropped 3.9 support

This is mainly because we'll likely release in
a month or so.

---------

Signed-off-by: Nick Papior <nickpapior@gmail.com>

Author: zerothi

.github/workflows/test.yaml

@@ -68,7 +68,7 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        python-version: ['3.9', '3.13']
+        python-version: ['3.10', '3.13']
         full-test: [true, false]
         minimal-dep: [true, false]
         os: [ubuntu-latest, macos-13, macos-latest]
@@ -119,7 +119,7 @@ jobs:
         fi
         # Test minimal versioning?
         if [[ "${{ matrix.minimal-dep }}" == "true" ]]; then
-          echo "sisl_install_packages='numpy==1.21.*' 'scipy==1.6.*' 'xarray==0.21.*'" >> $GITHUB_OUTPUT
+          echo "sisl_install_packages='numpy==1.22.*' 'scipy==1.8.*' 'xarray==0.21.*'" >> $GITHUB_OUTPUT
         else
           echo "sisl_install_packages=" >> $GITHUB_OUTPUT
         fi

.github/workflows/wheels.yaml

@@ -114,12 +114,6 @@ jobs:
           CIBW_BUILD: "cp310-*"
           CIBW_ARCHS: ${{ matrix.cibw_archs }}
 
-      - name: Build wheels for CPython 3.9
-        uses: pypa/cibuildwheel@v3.1.4
-        env:
-          CIBW_BUILD: "cp39-*"
-          CIBW_ARCHS: ${{ matrix.cibw_archs }}
-
       - name: Store artifacts
         uses: actions/upload-artifact@v4
         with:
@@ -184,7 +178,7 @@ jobs:
       - name: Python installation
         uses: actions/setup-python@v6
         with:
-          python-version: "3.9"
+          python-version: "3.10"
 
       - name: Wait for testpypi to catch up
         run: |
@@ -201,7 +195,7 @@ jobs:
             --index-url https://test.pypi.org/simple/ \
             --extra-index-url https://pypi.org/simple/ \
             "sisl[test,viz]==${{ env.SISL_VERSION }}" \
-            "numpy==1.21.*" "scipy==1.6.*" "xarray==0.21.*"
+            "numpy==1.22.*" "scipy==1.8.*" "xarray==0.21.*"
 
       - name: Install sisl + dependencies (arm)
         if: matrix.os == 'macos-14'

CMakeLists.txt

@@ -23,6 +23,8 @@ cmake_print_variables(CMAKE_INTERPROCEDURAL_OPTIMIZATION)
 cmake_print_variables(CMAKE_POSITION_INDEPENDENT_CODE)
 cmake_print_variables(CMAKE_INSTALL_PREFIX)
 cmake_print_variables(CMAKE_PREFIX_PATH)
+cmake_print_variables(CMAKE_SYSTEM_NAME)
+cmake_print_variables(CMAKE_CROSSCOMPILING)
 if(SKBUILD)
   message(STATUS "scikit-build-core: ${SKBUILD_CORE_VERSION}")
 endif()

changes/orphan.3.change.rst

@@ -0,0 +1 @@
+Dropped support for Python 3.9

docs/quickstart/install.rst

@@ -74,7 +74,7 @@ The above installation instructions installs the necessary dependencies
 to run sisl, so generally one shouldn't worry about getting correct
 packages etc. Here the more detailed requirements are listed.
 
-- `Python`_ 3.9 or above
+- `Python`_ 3.10 or above
 - `numpy`_
 - `scipy`_
 - `xarray`_

pyproject.toml

@@ -24,7 +24,7 @@ build-backend = "scikit_build_core.build"
 
 # Top most items are probably changed the most
 [project]
-requires-python = ">=3.9"
+requires-python = ">=3.10"
 
 name = "sisl"
 description = "Manipulation and post-processing of DFT output and creating tight-binding models for NEGF transport"
@@ -52,8 +52,8 @@ keywords = [
 # lower bound of dependencies.
 dependencies = [
     # We need npt.NDArray
-    "numpy>=1.21",
-    "scipy>=1.6",
+    "numpy>=1.22",
+    "scipy>=1.8",
     # I can't get it to work on py311 with older versions
     "pyparsing>=2.2.1",
     # xarray fixes get_loc in 0.21
@@ -73,9 +73,9 @@ classifiers = [
     "Intended Audience :: Developers",
     "License :: OSI Approved :: Mozilla Public License 2.0 (MPL 2.0)",
     "Programming Language :: C",
+    "Programming Language :: Cython",
     "Programming Language :: Fortran",
     "Programming Language :: Python",
-    "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",

src/sisl/io/tests/test_zipfile.py

@@ -11,21 +11,7 @@
 import sisl
 from sisl.io._zipfile import ZipPath
 
-# We will xfail all tests if the python version is <=3.9,
-# this is because sisl is expected to raise a RuntimeError
-# explaining that zipfile functionality requires Python 3.10 or newer.
-_is_old_python = sys.version_info < (3, 10)
-# 3.9 also prints very ugly errors when deleting a zipfile which make
-# it difficult to read the test output. So we patch the __del__ method.
-if _is_old_python:
-    zipfile.ZipFile.__del__ = lambda self: None
-
-
-@pytest.mark.xfail(
-    _is_old_python,
-    reason="Zip file functionality requires Python 3.10 or newer",
-    raises=RuntimeError,
-)
+
 def test_zipfile_preserved():
     """Test that the zipfile is preserved through the sile framework
 
@@ -48,11 +34,6 @@ def test_zipfile_preserved():
     assert fdf.file.root is f
 
 
-@pytest.mark.xfail(
-    _is_old_python,
-    reason="Zip file functionality requires Python 3.10 or newer",
-    raises=RuntimeError,
-)
 @pytest.mark.parametrize("specify_class", [True, False])
 @pytest.mark.parametrize("external_zip", [True, False])
 def test_zipfile_write_read(external_zip: bool, specify_class: bool):
@@ -97,11 +78,6 @@ def test_zipfile_write_read(external_zip: bool, specify_class: bool):
     assert np.allclose(read_geometry.cell, geometry.cell)
 
 
-@pytest.mark.xfail(
-    _is_old_python,
-    reason="Zip file functionality requires Python 3.10 or newer",
-    raises=RuntimeError,
-)
 @pytest.mark.parametrize("external_zip", [True, False])
 @pytest.mark.parametrize("from_fdf", [True, False])
 def test_zipfile_write_read_binary(external_zip: bool, from_fdf: bool):
@@ -145,11 +121,6 @@ def test_zipfile_write_read_binary(external_zip: bool, from_fdf: bool):
     assert np.allclose(H.tocsr().toarray(), read_H.tocsr().toarray())
 
 
-@pytest.mark.xfail(
-    _is_old_python,
-    reason="Zip file functionality requires Python 3.10 or newer",
-    raises=RuntimeError,
-)
 @pytest.mark.parametrize("external_zip", [True, False])
 @pytest.mark.parametrize("from_fdf", [True, False])
 def test_zipfile_write_read_cdf(external_zip: bool, from_fdf: bool):