Skip to content

Conversation

peterjc123
Copy link
Contributor

No description provided.

@peterjc123 peterjc123 requested review from soumith and fmassa October 16, 2019 06:04
Copy link
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

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

LGTM, but can you explain why you needed to change the links to use S3? To make things faster to download?

@fmassa
Copy link
Member

fmassa commented Oct 16, 2019

Oh, just saw the comments in #1447 (comment)

@fmassa fmassa merged commit 0d87817 into pytorch:master Oct 16, 2019
@peterjc123
Copy link
Contributor Author

CUDA links are permanent, but the official cuDNN links require authentication. So I uploaded the cuDNN binary to my own dropbox storage. It is quite small, so I can't host too many binaries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants