Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Adapt MNIST example to use the original lecun data files #97
base: master
Are you sure you want to change the base?
Adapt MNIST example to use the original lecun data files #97
Changes from all commits
3d39f38
8f6e858
109ae26
5100f35
e355d7d
0fa41f2
128856a
c2af380
8cea7ed
426a8a1
081e2fa
f104e50
726cb2d
9667486
e5f52b1
a5ed003
19c3521
b544b93
7012c5f
a88d710
0e1462d
bb98d46
5b19f1e
165b447
d386d4a
99e4cd6
f7b53b0
6ab97a4
517da92
cc547a5
24678b0
2f724dc
b0b671b
ccd2679
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
A new example is a great idea, but should probably be a separate PR from the other changes you are making.
Having them a separate make reviewing both PRs easier.
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.
Understood. In this case the changes are focused on making the examples run (which they now do, as you can see from the travis output). I think that I've got to a stage where the pieces are in place. If you want to improve the run scripts, I think that that would be good. I'd be happy to flesh out some of the documentation, if that would help.
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.
Not too sure it the examples should run in CI. Will have a look.