Skip to content
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

Downloading MNIST fails due to HTTP Error 503 #632

Closed
akihironitta opened this issue May 6, 2021 · 4 comments · Fixed by #633
Closed

Downloading MNIST fails due to HTTP Error 503 #632

akihironitta opened this issue May 6, 2021 · 4 comments · Fixed by #633
Assignees
Labels
bug Something isn't working datamodule Anything related to datamodules

Comments

@akihironitta
Copy link
Contributor

akihironitta commented May 6, 2021

🐛 Bug

http://yann.lecun.com/exdb/mnist/index.html often gets unavailable, and because of the issue, some of the tests fail in Bolts due to 503. This is a known issue and its patch will be included in torchvision==0.9.1.

See pytorch/vision#3549

To Reproduce

https://dev.azure.com/PytorchLightning/lightning%20Bolts/_build/results?buildId=7496

@akihironitta akihironitta added fix fixing issues... datamodule Anything related to datamodules labels May 6, 2021
@akihironitta akihironitta self-assigned this May 6, 2021
@akihironitta
Copy link
Contributor Author

As far as I investigated, there are two known issues about MNIST downloading:

For 403 error, we can fix it by setting the user agent to avoid the cloudflare filter as done in PL: Lightning-AI/pytorch-lightning#6357

For 503 error, we need to use a different URL than http://yann.lecun.com/exdb/mnist/ because the website is very unstable and its patch is included in torchvision>=0.9.1.

@oke-aditya
Copy link
Contributor

oke-aditya commented May 6, 2021

The patch is already in stable torchvision, it redirects to a new mirror.

Here is the new mirror from torchvision datasets

Here is the mirror link

https://ossci-datasets.s3.amazonaws.com/mnist/

I think the bug is fixed from torchvision side. The recommended fix is to use torchvision==0.9.1

@akihironitta
Copy link
Contributor Author

@oke-aditya Yes, I'm aware of the mirror link. However, I don't think fixing the version of torchvision at 0.9.1 only for this issue would be a reasonable fix as it requires users to update their torchvision.

@oke-aditya
Copy link
Contributor

oke-aditya commented May 6, 2021

Yep I don't think so that's the way to fix for a single use case / issue.

What possibly can be done is to copy paste dataset.mnist from torchvision, and just use it.

Maybe something like

pl_bolts.datasets.mnist instead from torchvision.

@Borda Borda added bug Something isn't working and removed fix fixing issues... labels Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working datamodule Anything related to datamodules
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants