-
Notifications
You must be signed in to change notification settings - Fork 323
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
binary mnist dm fix #377
binary mnist dm fix #377
Conversation
Codecov Report
@@ Coverage Diff @@
## master #377 +/- ##
==========================================
+ Coverage 81.18% 81.22% +0.03%
==========================================
Files 100 100
Lines 5715 5714 -1
==========================================
+ Hits 4640 4641 +1
+ Misses 1075 1073 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you for the swift action :]
@akihironitta it seems that the Dataset caching does not work as expected or we shall not download it if it already exists...
|
@Borda I'll dig into it :] |
Fixes dataset not found error mentioned in issue #353
Reran the notebook that produced the error and it's now working:
https://colab.research.google.com/drive/1PHwAm48SNojrYbQWPeihmsbWuH-h6KU8#scrollTo=Kh5dXKO4Tr8Y