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.

Sign up for GitHub

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

Jump to bottom

93% accuracy attained with a simple CNN #14

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

93% accuracy attained with a simple CNN #14

wants to merge 12 commits into from

Conversation

VicHofs
Copy link

@VicHofs VicHofs commented Feb 3, 2020

Hey, I created a basic CNN with TensorFlow and got 93% balanced accuracy. Added it to the benchmarks, feel free to add it to main if deemed relevant.

P.S.: added code for reference and edited benchmarks table.

VicHofs added 10 commits February 3, 2020 17:36
@VicHofs
Create conv-net tensorflow model 93%
6c76139 
Basic CNN model using TensorFlow 2.0
benchmarks at 93% balanced accuracy with base version
@VicHofs
Notebook format
a611f41 
Original Jupyter Notebook with outputs and benchmarks.
@VicHofs
conv-net TensorFlow model full code
fd3d574 
Full code not in Notebook format
@VicHofs
Update k49 conv-net.ipynb
94877d7
@VicHofs
Add notebook format
c19b637 
Jupyter Notebook format of code
@VicHofs
Delete k49 conv-net.ipynb
2b566f6
@VicHofs
Update k49 cnn notebook.ipynb
72d85d9
@VicHofs
Delete k49 cnn notebook.ipynb
9b829ce
@VicHofs
Add Notebook format
46e08b0 
Jupyter Notebook format with complete code
@VicHofs
Update README.md
2b063db
@VicHofs VicHofs requested a review from mxbi February 3, 2020 21:31
mxbi
mxbi requested changes Feb 16, 2020
View reviewed changes
Copy link
Member

@mxbi mxbi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! I'm happy to add your results to the table, but I think it would be best if your model was in a separate repo that we can link into, instead of in the root of this one.

@VicHofs
Copy link
Author

VicHofs commented Feb 18, 2020

Thanks for the PR! I'm happy to add your results to the table, but I think it would be best if your model was in a separate repo that we can link into, instead of in the root of this one.

Alright. Is there a preferred format I should follow in the repo?

VicHofs
VicHofs commented Feb 18, 2020
View reviewed changes
Copy link
Author

@VicHofs VicHofs left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove model files from pull request fork

@VicHofs
Copy link
Author

VicHofs commented Feb 18, 2020

Basic repo at https://github.com/VicHofs/kmnist49-CNN-model-with-TensorFlow-2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mxbi mxbi mxbi requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@VicHofs @mxbi