-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add CNN class for MNIST in cnn.py and import it in main.py #137
Conversation
Sandbox Executions
|
Apply Sweep Rules to your PR?
|
The command:
|
1 similar comment
The command:
|
🚀 Wrote ChangesDone. |
🚀 Wrote ChangesI wasn't able to make changes. This could be due to an unclear request or a bug in my code. |
Description
This PR adds a new Convolutional Neural Network (CNN) class in cnn.py to handle the MNIST dataset. The CNN class is imported into main.py to replace the existing Net class. The CNN class consists of two convolutional layers followed by two fully connected layers. The input to the model is reshaped appropriately before being passed through the model.
Summary of Changes
Please review and merge this PR. Thank you!
Fixes #9.
🎉 Latest improvements to Sweep:
💡 To get Sweep to edit this pull request, you can: