-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[Bug Report] Automatic download of MNIST dataset fails #2057
Comments
I didn't encounter this issue, so the dataset should still be available. Did you try to upgrade torchvision
|
Thanks for a prompt reply. In order to reproduce, you need to exec all cells in order on a clean slate Python 3 (Data Science) container - as it is described in the header of the Notebook. And, yes, I expect that upgrading torchvision will solve the problem (as I implicitly mentioned earlier by suggesting to upgrade the hardcoded versions of pip-installed packages in the Notebook), but just upgrading torchvision, as you suggested, will pull and upgrade all dependent packages including torch, which may break the training and deployment steps if one is not careful with specifying the framework_version (e.g. pytorch/pytorch#25214). I feel that a more careful fix might be needed. More info and the torchvision fix: pytorch/vision#1938 Alternatively: download MNIST from an alternative location, e.g. a public 'sagemaker-sample-files' S3 bucket (as, e.g., done here: https://github.com/aws/amazon-sagemaker-examples/blob/master/frameworks/pytorch/get_started_mnist_train.ipynb). |
The issue you linked is for torchvision 0.4.2 but torchvision 0.3.0 is still fine. I can still use to download mnist data. If we use download it from a public bucket, then we will need to convert it to the format that can be used by torchvsion.datasets.MNIST in the notebook as well as the script |
Resolved with #2091 |
Link to the notebook
https://github.com/aws/amazon-sagemaker-examples/blob/master/sagemaker-experiments/mnist-handwritten-digits-classification-experiment/mnist-handwritten-digits-classification-experiment.ipynb
Describe the bug
Automatic download of MNIST dataset fails.
[In]
[Out]
To Reproduce
Run all cells in order.
Possible reason of error
The error is probably due to the download links for MNIST at https://github.com/pytorch/vision/blob/master/torchvision/datasets/mnist.py#L33-L36 are hosted on yann.lecun.com and that server has moved under CloudFlare protection.
Possible fix
Update the hardcoded version of PyTorch in the Notebook to a newer Pytorch version, where the MNIST download issue was fixed. See also this: pytorch/vision#1938
The text was updated successfully, but these errors were encountered: