MAINT: Switch to pip for macOS jobs

Author: larsoner

.github/workflows/tests.yml

@@ -30,7 +30,7 @@ jobs:
           persist-credentials: false
       - uses: actions/setup-python@v6
         with:
-          python-version: '3.13'
+          python-version: '3.14'
       - uses: pre-commit/action@v3.0.1
       - run: pip install mypy numpy scipy vulture
       - run: mypy
@@ -77,11 +77,11 @@ jobs:
             python: '3.13'
             kind: conda
           - os: macos-latest  # arm64 (Apple Silicon): Sequoia
-            python: '3.13'
-            kind: mamba
+            python: '3.14'
+            kind: pip
           - os: macos-15-intel  # intel: Sequoia
             python: '3.13'
-            kind: mamba
+            kind: pip
           - os: windows-latest
             python: '3.11'
             kind: mamba
@@ -123,31 +123,16 @@ jobs:
         with:
           python-version: ${{ matrix.python }}
         if: startswith(matrix.kind, 'pip')
+        id: setup-python
+      # Workaround macOS path behavior with login shells (which puts system Python first)
+      - run: echo "export PATH=\"$(dirname ${{ steps.setup-python.outputs.python-path }}):$PATH\"" | tee -a ~/.bash_profile  # zizmor: ignore[template-injection]
+        if: startswith(matrix.kind, 'pip') && startswith(matrix.os, 'macos')
       # Python (if conda)
-      - name: Fixes for conda
-        run: |
-          # For some reason on Linux we get crashes
-          if [[ "$RUNNER_OS" == "Linux" ]]; then
-            sed -i "/numba/d" environment.yml
-          fi
-          # And on Windows and macOS PySide6.9.0 segfaults
-          if [[ "$RUNNER_OS" == "macOS" ]]; then
-            sed -i "" "s/  - PySide6 .*/  - PySide6 =6.9.2/g" environment.yml
-            sed -i "" "s/  - vtk .*/  - vtk =9.5.1/g" environment.yml
-
-          else
-            sed -i "s/  - PySide6 .*/  - PySide6 =6.9.2/g" environment.yml
-            sed -i "s/  - vtk .*/  - vtk =9.5.1/g" environment.yml
-            if [[ "$RUNNER_OS" == "Windows" ]]; then
-              echo "MNE_IS_OSMESA=true" | tee -a $GITHUB_ENV
-            fi
-          fi
-        if: matrix.kind == 'conda' || matrix.kind == 'mamba'
       - uses: mamba-org/setup-micromamba@v3
         with:
           environment-file: ${{ env.CONDA_ENV }}
           environment-name: mne
-          log-level: ${{ runner.debug == '1' && 'debug' || 'info' }}
+          log-level: 'info'
           create-args: >-
             python=${{ env.PYTHON_VERSION }}
             -v
@@ -162,6 +147,7 @@ jobs:
             **/pylock.ci-old.toml
           python-version: ${{ matrix.python }}
         if: matrix.kind == 'old'
+      - run: bash ./tools/github_actions_verify_python.sh
       - run: bash ./tools/github_actions_dependencies.sh
       - run: python ./tools/github_actions_check_old_env.py
         if: matrix.kind == 'old'

environment.yml

@@ -43,7 +43,7 @@ dependencies:
   - pyarrow
   - pybv
   - pymatreader
-  - PySide6 !=6.9.1
+  - PySide6 ==6.10.2
   - python-neo
   - python-picard >=0.4
   - pyvista >=0.43
@@ -61,7 +61,7 @@ dependencies:
   - trame
   - trame-vtk
   - trame-vuetify
-  - vtk >=9.2
+  - vtk ==9.6.1
   - xlrd
   - pip:
       - nest-asyncio2

tools/github_actions_dependencies.sh

@@ -3,6 +3,7 @@
 set -eo pipefail
 
 SCRIPT_DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )
+ONLY_BINARY_ARG="--only-binary=numpy,scipy,matplotlib,numba,llvmlite,antio"
 STD_ARGS="--progress-bar off --upgrade"
 INSTALL_ARGS="-e"
 if [ ! -z "$CONDA_ENV" ]; then
@@ -61,6 +62,6 @@ else
 	echo "::group::Installing MNE in development mode using pip"
 fi
 set -x
-python -m pip install $STD_ARGS $INSTALL_ARGS .$EXTRAS $GROUP_ARG
+python -m pip install $STD_ARGS $ONLY_BINARY_ARG $INSTALL_ARGS .$EXTRAS $GROUP_ARG
 set +x
 echo "::endgroup::"

tools/github_actions_verify_python.sh

@@ -0,0 +1,26 @@
+#!/bin/bash -ef
+
+set -eo pipefail
+
+GOT_PYTHON=$(which python)
+echo "Checking Python at:
+echo "  which python=${GOT_PYTHON}
+echo "for"
+echo "  MNE_CI_KIND=${MNE_CI_KIND}"
+if [[ "${MNE_CI_KIND}" == "conda" ]] || [[ "${MNE_CI_KIND}" == "mamba" ]]; then
+	WANT="micromamba/envs/mne"
+elif [[ "${MNE_CI_KIND}" == "old" ]]; then
+	WANT="mne-python/mne-python/.venv/bin"
+elif [[ "${MNE_CI_KIND}" == "pip" ]] || [[ "${MNE_CI_KIND}" == "pip-pre" ]]; then
+	WANT="hostedtoolcache/Python"
+else
+	echo "ERROR: Unrecognized MNE_CI_KIND=${MNE_CI_KIND}"
+	exit 1
+fi
+if [[ "${GOT_PYTHON}" != *"${WANT}"* ]]; then
+	echo "ERROR: Did not find \"${WANT}\" from PATH:"
+	tr ':' '\n' <<< "$PATH"
+	exit 1
+else
+	echo "OK: Found expected \"${WANT}\""
+fi

tools/hooks/update_environment_file.py

@@ -57,7 +57,9 @@ def split_dep(dep):
     # `environment.yaml` breaks the solver. 6.9.1 has a bug, and 6.9.2 needs newer
     # C deps that mean we need to upgrade VTK etc.
     if package_name == "PySide6":
-        version_spec = "!=6.9.1"
+        version_spec = "==6.10.2"
+    if package_name == "vtk":
+        version_spec = "==9.6.1"
     # rstrip output line in case `version_spec` == ""
     line = f"  - {package_name} {version_spec}".rstrip()
     # use pip for packages needing e.g. `platform_system` or `python_version` triaging