Skip to content

Commit

Permalink
try to fix CUDA
Browse files Browse the repository at this point in the history
- change step condition
- see where NiftyPET*cmake is
- exclude CUDA tests from ctest
  • Loading branch information
KrisThielemans committed Nov 25, 2023
1 parent 3bcc1e7 commit b32fc74
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion .github/workflows/build-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ jobs:
echo CC="$CC" >> $GITHUB_ENV
echo CXX="$CXX" >> $GITHUB_ENV
- if: ${{ ! matrix.cuda_version == '0' }}
- if: matrix.cuda_version != '0'
uses: Jimver/cuda-toolkit@v0.2.11
id: cuda-toolkit
with:
Expand Down Expand Up @@ -246,6 +246,7 @@ jobs:
export HMUDIR=${GITHUB_WORKSPACE}/mmr_hardwareumaps
python -m pip install "nipet>=2"
niftypet_cmake_prefix=$(python -c "from niftypet.nipet import cmake_prefix; print(cmake_prefix)")
ls -R "$niftypet_cmake_prefix"
echo NiftyPET_PREFIX="$niftypet_cmake_prefix" >> $GITHUB_ENV
fi
Expand Down Expand Up @@ -333,6 +334,10 @@ jobs:
if test ${BUILD_TYPE} = Debug; then
EXCLUDE="-E test_data_processor_projectors|test_export_array|test_ArcCorrection|test_blocks_on_cylindrical_projectors"
fi
if test ${{matrix.cuda_version}} != "0"
# No CUDA drivers on GitHub Actions
EXCLUDE="${EXCLUDE}|parallelproj|NiftyPET|test_blocks_on_cylindrical_projectors"
fi
ctest --output-on-failure -C ${BUILD_TYPE} ${EXCLUDE}
- name: recon_test_pack
Expand Down

0 comments on commit b32fc74

Please sign in to comment.