-
Notifications
You must be signed in to change notification settings - Fork 388
Enable integration tests with the latest PyTorch by installing csprng from the source #407
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
Changes from 6 commits
ebcbf9e
b87b45d
1d49bbf
455834e
6241d1a
8e9be1a
b0bf58b
3e43b57
04f21b1
a28f71c
4b69df1
42cb464
4ebadd0
78ee9e2
5febb14
c9e3526
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -25,9 +25,4 @@ If ($TORCH_VERSION -eq "1.8.0") { | |
| $TORCHVISION_VERSION="0.10.1" | ||
| } | ||
| pip install torch==$TORCH_VERSION+cpu torchvision==$TORCHVISION_VERSION+cpu -f https://download.pytorch.org/whl/torch_stable.html | ||
|
|
||
| If ($TORCH_VERSION -eq "1.8.0") { | ||
| pip install torchcsprng==$TORCHCSPRNG_VERSION+cpu -f https://download.pytorch.org/whl/torch_stable.html | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. L19 is now irrelevant. |
||
| } Else { | ||
| echo "No torchcsprng" | ||
| } | ||
| pip install "git+https://github.com/pytorch/csprng.git@cd8f2f670355f5a9b345dbe2cb58a52fb2c44d39#egg=torchcsprng" | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -35,9 +35,4 @@ pip install torch=="${TORCH_VERSION}" | |
| pip install torchvision==${TORCHVISION_VERSION} | ||
|
|
||
| # torchcsprng | ||
| if [ "$TORCH_VERSION" = "1.8.0" ] | ||
| then | ||
| pip install torchcsprng==${TORCHCSPRNG_VERSION} | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I suppose this means L22 is no longer required? |
||
| else | ||
| echo "No torchcsprng" | ||
| fi | ||
| pip install git+https://github.com/pytorch/csprng.git@cd8f2f670355f5a9b345dbe2cb58a52fb2c44d39#egg=torchcsprng | ||
|
||
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.
should this be cu102 since you are changing Cuda version to 10.2?