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

Root directory of Fashion MNIST #2602

Closed
ibayramli opened this issue Aug 21, 2020 · 1 comment Β· Fixed by #2614
Closed

Root directory of Fashion MNIST #2602

ibayramli opened this issue Aug 21, 2020 · 1 comment Β· Fixed by #2614

Comments

@ibayramli
Copy link

πŸ“š Documentation

Hi there,

The docs of FashionMNIST dataset says:

root (string) – Root directory of dataset where Fashion-MNIST/processed/training.pt and Fashion-MNIST/processed/test.pt exist.

It should instead be:

root (string) – Root directory of dataset where FashionMNIST/processed/training.pt and FashionMNIST/processed/test.pt exist.

since when I manually place processed/training.pt and processed/test.pt into a folder named Fashion-MNIST, I get a RuntimeError: Dataset not found. Turning Fashion-MNIST into FashionMNIST solves the issue. When download=True, the system also generates the files in a folder named FashionMNIST not Fashion-MNIST.

This is not an extremely important bug, but it may save some time in the future for some people experiencing bugs due to this.

@albanD albanD transferred this issue from pytorch/pytorch Aug 21, 2020
@pmeier
Copy link
Collaborator

pmeier commented Aug 21, 2020

Hi @ibayramli2001, thanks for the report. Do you want to send a PR fixing this?

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

Successfully merging a pull request may close this issue.

2 participants