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

The CNN model does not reach 93% accuracy at all #1

Open
minqz2009 opened this issue Apr 1, 2020 · 2 comments
Open

The CNN model does not reach 93% accuracy at all #1

minqz2009 opened this issue Apr 1, 2020 · 2 comments

Comments

@minqz2009
Copy link

The CNN model does not reach 93% accuracy at all. This model could only achieve an accuracy around 50~60%. The reason the accuracy from the output of the notebook in this repo is because, in the test function, they forgot to reinitialize the count and total.
image
So that 80% of test accuracy is dominant by the train accuracy.
Below is the real accuracy of the CNN model provided in this repo:
image

@goncinious
Copy link
Owner

Thanks for pointing this out @minqz2009! I'll correct it and update the repo as soon as I have time to improve it.

@minqz2009
Copy link
Author

No worries mate

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

No branches or pull requests

2 participants