Skip to content

Commit

Permalink
revert back to macos git CI
Browse files Browse the repository at this point in the history
  • Loading branch information
souryadey committed Oct 14, 2023
1 parent e76c444 commit f9f764d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/pytest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ on: push

jobs:
test:
runs-on: ubuntu-latest
runs-on: macos-latest
timeout-minutes: 10

steps:
Expand Down

1 comment on commit f9f764d

@souryadey
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The default Git runner for ubuntu-latest does not come with CuDA. Unfortunately, the default pip install torch on Ubuntu installs the CuDA version, which gives the error OSError: libcufft.so.11: cannot open shared object file: No such file or directory when running pytest in CI. This can probably be fixed by installing the CPU only version via pip install torch --index-url https://download.pytorch.org/whl/cpu (or even a version with different CuDA via pip install torch --index-url https://download.pytorch.org/whl/cu118), but I don't know how to give pip install options to poetry (the poetry documentation is not clear on this). Hence, I'm switching the tests to run on MacOS for now.

Note that I ran all the tests via pytest on an Ubuntu machine (Galois Razor), and they all pass. However, this was after I installed the proper version of CuDA on this Ubuntu machine by passing options to pip install. So, I'm confident that the code is fine and will run properly on Linux (or Ubuntu at least), but I have to make the Github runner Mac to make CI pass due to peculiarities of poetry and Git runners.

Please sign in to comment.