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

Automatically download CORe50 instead of requiring user to download it. #30

Closed
AndreaCossu opened this issue May 11, 2020 · 3 comments · Fixed by #32
Closed

Automatically download CORe50 instead of requiring user to download it. #30

AndreaCossu opened this issue May 11, 2020 · 3 comments · Fixed by #32
Assignees
Labels
bug Something isn't working

Comments

@AndreaCossu
Copy link
Collaborator

Unlike other datasets like MNIST, CORe50 must be explicitly downloaded from the user. It would be better to have the same interface for all datasets.
Moreover, this behavior also breaks the script examples/simple_core50.py, since the provided data folder does not exist.

@AndreaCossu AndreaCossu added the bug Something isn't working label May 11, 2020
@AndreaCossu AndreaCossu self-assigned this May 11, 2020
@AndreaCossu
Copy link
Collaborator Author

AndreaCossu commented May 11, 2020

@vlomonaco can you confirm that the only files needed for CORe50 are the ones listed in the data loader description? Namely: core50_128x128, paths.pkl, LUP.pkl, labels.pkl, core50_imgs.npz. I assume that core50_128x128 refers to the unzipped folder, not to the zip file.

@vlomonaco
Copy link
Member

yes, I think so! I think the preload of the dataset in ram should be disabled by default... so core50_imgs.npz is not really needed!

@AndreaCossu
Copy link
Collaborator Author

Ok, I will download also that file, just in case some user wants to use the preload option.

@AndreaCossu AndreaCossu linked a pull request May 11, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants