Merge pull request #892 from AlexVCaron/feat/github_runner

[WIP] Kill Jenkins

.github/workflows/freeze_requirements.yml

@@ -7,7 +7,7 @@ on:
 
 jobs:
   freeze_requirements:
-    runs-on: ubuntu-latest
+    runs-on: scilus-runners
     steps:
       - 
         name: Checkout scilpy

.github/workflows/test.yml

@@ -0,0 +1,70 @@
+name: Scilpy test suite
+
+on:
+  push:
+    branches:
+      - master
+  pull_request_target:
+    branches:
+      - master
+
+env:
+  MPLBACKEND: agg
+  OPENBLAS_NUM_THREADS: 1
+
+jobs:
+  test:
+    runs-on: scilus-runners
+    steps:
+      - name: Checkout repository
+        uses: actions/[email protected]
+
+      - name: Fetch python version from repository
+        id: python-selector
+        run: echo "python-version=$(cat .python-version)" >> $GITHUB_OUTPUT
+
+      - name: Set up Python for Scilpy
+        uses: actions/[email protected]
+        with:
+          python-version: ${{ steps.python-selector.outputs.python-version }}
+          cache: 'pip'
+
+      - name: Install non-python dependencies
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y \
+            build-essential \
+            curl \
+            git \
+            libblas-dev \
+            liblapack-dev \
+            libfreetype6-dev
+
+      - name: Install Scilpy
+        run: |
+          python -m pip install --upgrade pip setuptools wheel
+          python -m pip install -e .
+
+      - name: Run tests
+        run: |
+          export C_INCLUDE_PATH=$pythonLocation/include/python${{ steps.python-selector.outputs.python-version }}:$C_INCLUDE_PATH
+          pytest --cov-report term-missing:skip-covered
+
+      - name: Upload coverage reports to Codecov
+        uses: codecov/codecov-action@v3
+        env:
+          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+        with:
+          flags: unittests
+          name: scilpy-unittests-${{ github.run_id }}
+          verbose: true
+          directory: .test_reports/
+          fail_ci_if_error: true
+          root_dir: $GITHUB_WORKSPACE/scilpy/
+
+      - name: Upload test reports and coverage to artifacts
+        uses: actions/[email protected]
+        with:
+          name: test-reports
+          path: |
+            .test_reports/*

MANIFEST.in

@@ -3,6 +3,7 @@ include LICENSE
 include requirements.txt
 include .python-version
 
+recursive-include data/LUT *
 recursive-include scilpy *.c
 recursive-include scilpy *.cpp
 recursive-include scilpy *.pyx

scripts/scil_tractogram_filter_by_anatomy.py

@@ -49,7 +49,7 @@
 import json
 import logging
 import os
-import pkg_resources
+import importlib.resources as resources
 
 from dipy.io.streamline import save_tractogram
 from dipy.io.utils import is_header_compatible
@@ -142,9 +142,8 @@ def load_wmparc_labels():
     Load labels dictionary of different parcellations from the
     Desikan-Killiany atlas
     """
-    resource_package = pkg_resources.get_distribution('scilpy').location
-    labels_path = os.path.join(
-        resource_package, 'data/LUT/dk_aggregate_structures.json')
+    lut_package = resources.files('data').joinpath('LUT')
+    labels_path = lut_package.joinpath('dk_aggregate_structures.json')
     with open(labels_path) as labels_file:
         labels = json.load(labels_file)
     return labels