Skip to content

Commit 56276f2

Browse files
authored
Remove 3rd-party dependencies from Python coverage workflow
CI Action fails when trying to set up micromamba for FSL and ANTs. So for now, only get faithful test coverage of commands that have zero third-party neuroimaging dependencies; these can all be added together at a later date.
1 parent 668560c commit 56276f2

1 file changed

Lines changed: 1 addition & 21 deletions

File tree

.github/workflows/weekly_python_coverage.yml

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -30,26 +30,6 @@ jobs:
3030
with:
3131
cmakeVersion: '3.22.1'
3232

33-
- name: Set up neuroimaging dependencies
34-
uses: mamba-org/setup-micromamba@v2
35-
with:
36-
micromamba-version: latest
37-
environment-name: neuro
38-
create-args: >-
39-
-c conda-forge
40-
python>=3.8
41-
fsl
42-
fsl-eddy-cpu
43-
ants
44-
cache-environment: true
45-
init-shell: bash
46-
47-
- name: Configure external tool environment
48-
run: |
49-
echo "FSLDIR=$CONDA_PREFIX" >> $GITHUB_ENV
50-
echo "FSLOUTPUTTYPE=NIFTI_GZ" >> $GITHUB_ENV
51-
echo "ANTSPATH=$CONDA_PREFIX/bin" >> $GITHUB_ENV
52-
5333
- name: Install Python coverage tools
5434
run: pip install coverage deepatropos
5535

@@ -91,7 +71,7 @@ jobs:
9171
echo "COVERAGE_PYTHONPATH=/tmp/pycov${PYTHONPATH:+:$PYTHONPATH}" >> $GITHUB_ENV
9272
9373
- name: Run Python script tests
94-
# Tests requiring AFNI (3dautomask), HD-BET, or SynthStrip will fail;
74+
# Tests requiring any external neuroimaging software will fail;
9575
# their partial execution still contributes coverage for reached code paths.
9676
# All tests share the same working directory (script_test_data root) and
9777
# write to overlapping tmp* paths, so parallelism is explicitly disabled.

0 commit comments

Comments
 (0)