Update workflows

Author: lorenzotomada

.github/workflows/docs.yml

@@ -17,27 +17,24 @@ jobs:
   docs:
     name: Documentation
     runs-on: ubuntu-latest
-    container:
-      image: cupy/cupy:v13.4.0
 
     steps:
     - uses: actions/checkout@v3
 
     - name: Clean previous builds
       run: rm -rf build dist *.egg-info
 
-    - name: Set up git
-      run: apt-get update && apt-get install git -y
-
     - name: Install system dependencies
       run: |
-        apt-get update && apt-get install -y python3-venv python3-pip
+        sudo apt-get update
+        sudo apt-get install -y python3.12 python3.12-venv python3-pip git
 
-    - name: Install additional dependencies
+    - name: Create virtual environment and install package
       run: |
-        python3.10 -m venv venv
+        python3.12 -m venv venv
         source venv/bin/activate
-        python3.10 -m pip install --no-cache-dir .
+        python3.12 -m pip install --upgrade pip
+        python3.12 -m pip install --no-cache-dir .
       shell: bash -e {0}
 
     - name: Generate documentation
@@ -46,7 +43,7 @@ jobs:
         cd docs
         make html
       shell: bash -e {0}
-    
+
     - name: Deploy
       if: success()
       uses: peaceiris/actions-gh-pages@v3

.github/workflows/tests.yml

@@ -16,35 +16,43 @@ on:
 
 jobs:
   test:
-    name: Testing using pytest
+    name: Testing on Python ${{ matrix.python-version }}
     runs-on: ubuntu-latest
-    container:
-      image: cupy/cupy:v13.4.0
+
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ['3.10', '3.11', '3.12']
 
     steps:
     - uses: actions/checkout@v3
 
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v5
+      with:
+        python-version: ${{ matrix.python-version }}
+
     - name: Clean previous builds
       run: rm -rf build dist *.egg-info
 
     - name: Install system dependencies
       run: |
-        apt-get update && apt-get install -y python3.10-venv python3-pip cmake
+        sudo apt-get update
+        sudo apt-get install -y cmake
 
-    - name: Install additional dependencies
+    - name: Install Python dependencies
       run: |
-        python3.10 -m venv venv
+        python -m venv venv
         source venv/bin/activate
-        python3.10 -m pip install --no-cache-dir .
-      shell: bash -e {0}
+        python -m pip install --upgrade pip
+        pip install --no-cache-dir .
+      shell: bash
 
-    - name: Test with pytest
+    - name: Run tests with pytest
       run: |
         source venv/bin/activate
         mkdir -p build
         cmake -B build -S .
-        cd build
-        make
-        cd ..
-        python3.10 -m pytest
-      shell: bash -e {0}
+        cmake --build build
+        pytest
+      shell: bash

README.md

@@ -12,3 +12,5 @@ TO DO:
 4) Accuracy vs efficiency
 5) Script to run on Ulysses
 6) Add missing tests
+7) Possibly delete eigenvalues\_np and so on, which I inserted because I thought it was not going to be necessary to compute the eigenvectors, making these wrappers useless
+8) Write missing documentation

setup.py

@@ -48,7 +48,11 @@ def build_extension(self, ext):
     ext_modules=[CMakeExtension("pyclassify.QR_cpp")],
     packages=find_packages(where="src/"),
     package_dir={"": "src/"},
-    package_data={"pyclassify": ["QR_cpp.cpython-312-x86_64-linux-gnu.so"]},
+    package_data={
+        "pyclassify": [
+            f"QR_cpp.cpython-{sys.version_info[0]}{sys.version_info[1]}-x86_64-linux-gnu.so"
+        ]
+    },
     include_package_data=True,
     cmdclass={"build_ext": CMakeBuild},
     zip_safe=False,