Skip to content

Commit

Permalink
Run unit tests on MacOS in CI (#83)
Browse files Browse the repository at this point in the history
* Run unit tests on MacOS in CI

Signed-off-by: Fabrice Normandin <[email protected]>

* Remove specific cuda library reqs from 'gpu' extra

Signed-off-by: Fabrice Normandin <[email protected]>

* Add a skipif mark on test for MacOS in CI

Signed-off-by: Fabrice Normandin <[email protected]>

---------

Signed-off-by: Fabrice Normandin <[email protected]>
  • Loading branch information
lebrice authored Nov 7, 2024
1 parent e8514f6 commit bc5723f
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 44 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ jobs:
strategy:
max-parallel: 4
matrix:
platform: [ubuntu-latest]
platform: [ubuntu-latest, macos-latest]
python-version: ["3.10"]
steps:
- uses: actions/checkout@v4
Expand Down
4 changes: 4 additions & 0 deletions project/main_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -223,6 +223,10 @@ def test_example_experiment_defaults(experiment_config: Config) -> None:
)


@pytest.mark.skipif(
IN_GITHUB_CI and sys.platform == "darwin",
reason="TODO: Getting a 'MPS backend out of memory' error on the Github CI. ",
)
@pytest.mark.parametrize(
command_line_overrides.__name__,
[
Expand Down
15 changes: 1 addition & 14 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -61,20 +61,7 @@ docs = [
"mkdocs-macros-plugin>=1.0.5",
]
gpu = [
"jax[cuda12]>=0.4.31",
"nvidia-cublas-cu12==12.1.3.1",
"nvidia-cuda-cupti-cu12==12.1.105",
"nvidia-cuda-nvcc-cu12==12.6.68",
"nvidia-cuda-nvrtc-cu12==12.1.105",
"nvidia-cuda-runtime-cu12==12.1.105",
"nvidia-cudnn-cu12==9.1.0.70",
"nvidia-cufft-cu12==11.0.2.54",
"nvidia-curand-cu12==10.3.2.106",
"nvidia-cusolver-cu12==11.4.5.107",
"nvidia-cusparse-cu12==12.1.0.106",
"nvidia-nccl-cu12==2.20.5",
"nvidia-nvjitlink-cu12==12.6.68",
"nvidia-nvtx-cu12==12.1.105",
"jax[cuda12]>=0.4.31"
]

[build-system]
Expand Down
29 changes: 0 additions & 29 deletions uv.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit bc5723f

Please sign in to comment.