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

Code review by Yang Qiao #1

Open
JesusJoey opened this issue Nov 3, 2018 · 0 comments
Open

Code review by Yang Qiao #1

JesusJoey opened this issue Nov 3, 2018 · 0 comments

Comments

@JesusJoey
Copy link

JesusJoey commented Nov 3, 2018

Hi. I can see from your repository that you have done a lot of of researches on CNN. You implemented some models like AlexNet and basic CNN models. This is very helpful for further study. Also I think you should compare between these models instead of just running them. You can tell that in which way AlexNet is better or not. You can give us more details in the Readme. For instance, what is your dataset for training and how you implement the preprocessing? Which model did you choose in your training.py. These will make your work more readable. Hope you are doing well with your project.
Cheers!

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

1 participant