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

fix bug of canot load EMNIST dataset #5356 #5401

Closed

Conversation

nonameR9077
Copy link
Contributor

Description

The current MNIST URL - https://www.itl.nist.gov/iaui/vip/cs_links/EMNIST/gzip.zip is outdated which gives the NonMatchingChecksumError whenever tfds.image_classification.EMNIST is called.
Simply changed to working URL - https://biometrics.nist.gov/cs_links/EMNIST/gzip.zip which gives the same gzip.zip file.
This was first issued by davidshen84 here and found the working URL. Hurray for davidshen84!

just fixed to working URL
@ccl-core ccl-core added the copybara-import Internal label for PR management label May 8, 2024
@ccl-core
Copy link
Collaborator

ccl-core commented May 8, 2024

Hi @nonameR9077 , thanks for your contribution!

Your PR is under review and will be merged soon.

For future reference notice that, when changing download links:

@ccl-core ccl-core closed this May 8, 2024
@ccl-core ccl-core self-assigned this May 8, 2024
copybara-service bot pushed a commit that referenced this pull request May 8, 2024
@nonameR9077
Copy link
Contributor Author

Checksums / versions / release notes, copy.
thank you for your fast review!

@ccl-core
Copy link
Collaborator

ccl-core commented May 8, 2024

Thank you for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
copybara-import Internal label for PR management
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants