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 for issue 52: cpu mode epochs #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix for issue 52: cpu mode epochs #76

wants to merge 2 commits into from

Conversation

TTN-
Copy link

@TTN- TTN- commented Mar 15, 2017

Applying fix for issues:
#52
#75

@TTN-
Copy link
Author

TTN- commented Mar 15, 2017

EDIT: ignore that last commit for now, its semi broken.

I've just added one more commit to my fork, not sure if you'd like to have it in yours too.
I find that the --dataset size parameter isn't listened to at all if the dataset has already been generated. So if I run it once with --dataset 1000, then later again with --dataset 80000 later, I will still actually be using the 1000 dataset. Generating the dataset takes only a fraction of the time needed to train a network, so it is easiest to fix this by simply generating the dataset on each run as a workaround. Like I said maybe you dont want to include it here, but its there if you like it..
Just want to say though, this project is amazing, I'm toying around with it a lot. Very interesting to test on.

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

Successfully merging this pull request may close these issues.

2 participants