Skip to content

Commit

Permalink
Run pip install unconditionally in CI workflow (#1626)
Browse files Browse the repository at this point in the history
The cache is not guaranteed to be complete or up to date. The CI workflow is currently not running any of the Python tests because of a missing Numpy package.
  • Loading branch information
ahoenselaar authored Jun 23, 2021
1 parent 8903b50 commit 4d27231
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/build-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,6 @@ jobs:
restore-keys: ${{ runner.os }}-py${{ matrix.python-version }}-mpi-${{ matrix.enable-mpi }}-pip-

- name: Install Python dependencies
if: steps.cache.outputs.cache-hit != 'true'
run: pip install -r python/requirements.txt

- name: Install mpi4py
Expand Down Expand Up @@ -161,4 +160,4 @@ jobs:
if: failure()
with:
name: py${{ matrix.python-version }}-tests-mpi-${{ matrix.enable-mpi }}-log
path: ${{ github.workspace }}/build/meep-${{ env.MEEP_VERSION }}/_build/sub/python/test-suite.log
path: ${{ github.workspace }}/build/meep-${{ env.MEEP_VERSION }}/_build/sub/python/test-suite.log

0 comments on commit 4d27231

Please sign in to comment.