-
Notifications
You must be signed in to change notification settings - Fork 505
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix torchvision installation in tpu ci test setup script #6704
Conversation
Cannot find a way to read the When doing To rule out the fact that we cannot change env var in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also mirror the fix to the GHA CI if it works? https://github.com/pytorch/xla/blob/master/.github/workflows/tpu_ci.yml
The commit pin will be more readily available.
475deda
to
c5b1a6c
Compare
…tallation from tpu ci dockerfile to tpu ci job yaml
I had a few attempts in the previous commits fixing the GHA TPU CI, but failed to do so, let's land the fix for installing torchvision properly for the old TPU CI. |
GHA TPU CI fixed in #6730 |
Currently in TPU CI, we are installing torch nightly whl before running tests, however, in TPU CI, torch_xla is built against PyTorch HEAD. This would cause some compatibility issues. (e.g. Symbol mismatching in .so)
torchaudio is not needed for CI, remove torchaudio installation in the setup script as well.
Instead of installing nightly torchvision, use the torchvision version pinned in PyTorch src folder.