-
Notifications
You must be signed in to change notification settings - Fork 4.2k
some improvements to the make download(#384) #390
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
Conversation
Deploy preview for pytorch-tutorials-preview ready! Built with commit b037aef https://deploy-preview-390--pytorch-tutorials-preview.netlify.com |
Here's the
|
Hey Junghwan, thanks for your work on this. I'm going to test the build on a local machine and we'll look at merging this next week. |
Thanks. Please let me know if any issue occurs while testing. 🤞 |
Junghwan, I've been having trouble with getting this build to work on my test server. (Not anything in your code, just an issue with my server.) Could you rebase and we'll take a look at the check logs from the PR? I didn't see any issues as far as my build got. Thanks. |
Done. I hope there's no problem anymore. |
Sorry for missing data dir |
some improvements to the make download(pytorch#384)
I've made the following improvements in
download
:curl
commands towget
.wget
commands with the-N
option. Therefore, download and compare the timestamp.$(DATADIR)
directory (using-P
option).>null
) has been replaced by the-q
(quiet) option tounzip
.sudo
/chmod
.data
) have been deleted.Please review this PR.
Thanks.