Skip to content

Commit

Permalink
naive attempt at fixing HDF5 (#100)
Browse files Browse the repository at this point in the history
* naive attempt at fixing HDF5

* sanity check that hdf5 install needed

* Revert "sanity check that hdf5 install needed"

It is indeed needed.

This reverts commit e4db17d.

* remove adding hdf to path
  • Loading branch information
alessandrofelder committed Apr 30, 2024
1 parent cd1079a commit b5f62ef
Showing 1 changed file with 12 additions and 6 deletions.
18 changes: 12 additions & 6 deletions .github/workflows/test_and_deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,20 +36,26 @@ jobs:
# Run tests on ubuntu across all supported versions
python-version: ["3.9", "3.10"]
os: [ubuntu-latest]
# Include at least one MacOS and Windows test
# Include a Windows test and old/new Mac runs
include:
- os: macos-latest
python-version: "3.10"
- os: windows-latest
python-version: "3.10"
- os: macos-13
python-version: "3.10"
- os: macos-latest
python-version: "3.10"
- os: windows-latest
python-version: "3.10"
steps:
# Cache the tensorflow model so we don't have to remake it every time
- name: Cache tensorflow model
uses: actions/cache@v3
with:
path: "~/.cellfinder"
key: models-${{ hashFiles('~/.cellfinder/**') }}

# install additional Macos dependencies
- name: install HDF5 libraries (needed on M1 Macs only)
if: matrix.os == 'macos-latest'
run: |
brew install hdf5
# Run tests
- uses: neuroinformatics-unit/actions/test@v2
with:
Expand Down

0 comments on commit b5f62ef

Please sign in to comment.