Skip to content

Commit

Permalink
GHA: use clang for amici-heavy tests (#1198)
Browse files Browse the repository at this point in the history
Should be faster. Also use all cores for building amici models.

Saves 2-3 minutes for the petab test suite.
  • Loading branch information
dweindl authored Nov 21, 2023
1 parent bcdbd55 commit 2e49b71
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 2 deletions.
10 changes: 10 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ on:
# run Monday and Thursday at 03:42 UTC
- cron: '42 3 * * MON,THU'

env:
# use all available cores for compiling amici models
AMICI_PARALLEL_COMPILE: ""

# jobs
jobs:
base:
Expand Down Expand Up @@ -40,6 +44,9 @@ jobs:
- name: Run tests
timeout-minutes: 25
run: tox -e base
env:
CC: clang
CXX: clang++

- name: Coverage
uses: codecov/codecov-action@v3
Expand Down Expand Up @@ -133,6 +140,9 @@ jobs:
- name: Run tests
timeout-minutes: 35
run: tox -e petab
env:
CC: clang
CXX: clang++

- name: Coverage
uses: codecov/codecov-action@v3
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/install_deps.sh
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ for par in "$@"; do
brew install swig hdf5 libomp
else
sudo apt-get install \
swig libatlas-base-dev libhdf5-serial-dev
clang libatlas-base-dev libhdf5-serial-dev libomp-dev swig
fi
;;

Expand Down
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ envlist =
# Base-environment

[testenv]
passenv = AMICI_PARALLEL_COMPILE
passenv = AMICI_PARALLEL_COMPILE,CC,CXX

# Sub-environments
# inherit settings defined in the base
Expand Down

0 comments on commit 2e49b71

Please sign in to comment.