Refactor: Remove Cython - Reduce tech debt

.github/workflows/pytest.yml

@@ -1,4 +1,7 @@
 name: Tests
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
 
 on:
   # Allow to manually trigger through github API
@@ -8,13 +11,11 @@ on:
   push:
     branches:
       - main
-      - development
 
   # Triggers with push to a pr aimed at master
   pull_request:
     branches:
       - main
-      - development
 
   schedule:
     # Every day at 7AM UTC
@@ -31,16 +32,6 @@ env:
     --durations=20
     -v
 
-  # Version to run code-cov on
-  # NOTE: These are only acessible inside a jobs **steps** and not in the job setup,
-  #   Hence, some of these varialbes are copied and are just here for reference
-  #
-  code-cov-active: true  # Copied in job setup
-  code-cov-os: ubuntu-latest  # Copied in job setup
-  code-cov-python-version: "3.8"
-  code-cov-args: >-
-    --cov=ConfigSpace
-    --cov-report=xml
 
 jobs:
 
@@ -56,15 +47,21 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        python-version: ["3.7", "3.8", "3.9", "3.10", "3.11"]
+        python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
         os: ["ubuntu-latest", "macos-latest", "windows-latest"]
+        exclude:
+          # These don't seem to be available anymore with setup-python?
+          - python-version: "3.8"
+            os: "macos-latest"
+          - python-version: "3.9"
+            os: "macos-latest"
 
     steps:
     - name: Checkout
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4
 
     - name: Setup Python
-      uses: actions/setup-python@v2
+      uses: actions/setup-python@v5
       with:
         python-version: ${{ matrix.python-version }}
 
@@ -73,136 +70,7 @@ jobs:
         python -m pip install --upgrade pip
         python -m pip install -e ".${{ env.extra-requires }}"
 
-    - name: Store git status
-      id: status-before
-      shell: bash
-      run: |
-        echo "::set-output name=BEFORE::$(git status --porcelain -b)"
-
     - name: Tests
       timeout-minutes: 45
       run: |
         pytest ${{ env.pytest-args }} ${{ env.test-dir }}
-
-    - name: Check for files left behind by test
-      run: |
-        before="${{ steps.status-before.outputs.BEFORE }}"
-        after="$(git status --porcelain -b)"
-        if [[ "$before" != "$after" ]]; then
-            echo "git status from before: $before"
-            echo "git status from after: $after"
-            echo "Not all generated files have been deleted!"
-            exit 1
-        fi
-
-  # Testing with conda
-  conda-tests:
-    name: conda-${{ matrix.python-version }}-${{ matrix.os }}
-    runs-on: ${{ matrix.os }}
-    defaults:
-      run:
-        shell: bash -l {0}  # Default to using bash on all and load (-l) .bashrc which miniconda uses
-
-    strategy:
-      fail-fast: false
-      matrix:
-        python-version: ["3.7", "3.8", "3.9", "3.10", "3.11"]
-        os: ["ubuntu-latest", "macos-latest", "windows-latest"]
-
-    steps:
-    - name: Checkout
-      uses: actions/checkout@v2
-
-    - name: Conda install
-      uses: conda-incubator/setup-miniconda@v2
-      with:
-        auto-update-conda: true
-        python-version: ${{ matrix.python-version }}
-
-    - name: Install ${{ env.package-name }}
-      run: |
-        python -V
-        python -m pip install --upgrade pip
-        python -m pip install wheel
-        python -m pip install -e ".${{ env.extra-requires }}"
-
-    - name: Tests
-      timeout-minutes: 45
-      run: |
-          pytest ${{ env.pytest-args }} ${{ env.test-dir }}
-          #
-  # Testing a dist install
-  dist-test:
-    name: dist-${{ matrix.python-version }}-${{ matrix.os }}
-
-    runs-on: ${{ matrix.os }}
-    defaults:
-      run:
-        shell: bash
-
-    strategy:
-      fail-fast: false
-      matrix:
-        python-version: ["3.7", "3.8", "3.9", "3.10", "3.11"]
-        os: ["ubuntu-latest", "macos-latest", "windows-latest"]
-
-    steps:
-    - name: Checkout
-      uses: actions/checkout@v2
-
-    - name: Setup Python
-      uses: actions/setup-python@v2
-      with:
-        python-version: ${{ matrix.python-version }}
-
-    - name: Create sdist
-      id: sdist
-      run: |
-        python -m pip install --upgrade pip build
-        python -m build --sdist
-        echo "sdist_name=$(ls -t dist/${{ env.package-name }}-*.tar.gz | head -n 1)" >> "$GITHUB_ENV"
-
-    - name: Install ${{ env.package-name }}
-      run: |
-        python -m pip install ${{ env.sdist_name }}${{ env.extra-requires }}
-
-    - name: Tests
-      timeout-minutes: 45
-      run: |
-        pytest ${{ env.pytest-args }} ${{ env.test-dir }}
-
-
-  # Testing with codecov coverage uploaded
-  codecov-test:
-    name: codecov-test
-
-    runs-on: "ubuntu-latest"
-    defaults:
-      run:
-        shell: bash
-
-    steps:
-    - name: Checkout
-      uses: actions/checkout@v2
-
-    - name: Setup Python
-      uses: actions/setup-python@v2
-      with:
-        python-version: ${{ env.code-cov-python-version }}
-
-    - name: Install ${{ env.package-name }}
-      run: |
-        python -m pip install --upgrade pip
-        python -m pip install wheel
-        python -m pip install -e ".${{ env.extra-requires }}"
-
-    - name: Tests
-      timeout-minutes: 45
-      run: |
-        pytest ${{ env.pytest-args }} ${{ env.code-cov-args }} ${{ env.test-dir }}
-
-    - name: Upload coverage
-      uses: codecov/codecov-action@v2
-      with:
-        fail_ci_if_error: true
-        verbose: true

.gitignore

@@ -9,6 +9,7 @@ docs/examples/*
 # C extensions
 *.so
 *.c
+*.jukit
 
 # Packages
 *.egg
@@ -74,4 +75,5 @@ prof/
 .venv
 
 # Running pre-commit seems to generate these
-.mypy_cache
+.mypy_cache
+*.speedscope

.pre-commit-config.yaml

@@ -38,32 +38,15 @@ repos:
     hooks:
       - id: mypy
         files: '^ConfigSpace/.*\.py$'
+        additional_dependencies:
+          - "types-pyyaml"
         args:
           - "--no-warn-return-any" # Disable this because it doesn't know about 3rd party imports
           - "--ignore-missing-imports"
           - "--show-traceback"
-      - id: mypy
-        files: '^test/.*\.py$'
-        args:
-          - "--no-warn-return-any" # Disable this because it doesn't know about 3rd party imports
-          - "--ignore-missing-imports"
-          - "--show-traceback"
-          - "--disable-error-code"
-          - "no-untyped-def"
-
-  - repo: https://github.com/psf/black
-    rev: 23.3.0
-    hooks:
-      - id: black
-        args: ["--config=pyproject.toml"]
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.0.263
+    rev: v0.4.0
     hooks:
       - id: ruff
-        args: ["--fix", "ConfigSpace", "test"]
-  - repo: https://github.com/MarcoGorelli/cython-lint
-    rev: v0.15.0
-    hooks:
-      - id: cython-lint
-        args: [--ignore=E501]
-      - id: double-quote-cython-strings
+        args: [--fix, --exit-non-zero-on-fix, --no-cache]
+      - id: ruff-format