-
Notifications
You must be signed in to change notification settings - Fork 1
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 notebook on ConvMixer implementation #28
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the notebook name must be completely in lower case, you must also update it in the csv, see rules
changes made in another commit called "change the name of the notebook and the csv name". |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems I didn't make myself understood, check the comments again, please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I said in the other comment, the name of the .ipynb must be completely lowercase
changes made in another commit called "Change the Csv name to lowercase". |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name of the .ipynb FILEE must be completely in lowercase 😭 (ConvMixer.ipynb -> convmixer.ipynb)
changes made in another commit called "only lower case". |
The notebooks table data file has made changes that now make conflicts with the changes that you proposed. Please fix it. |
changes made in another commit called "conflicts resolved with main". |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
implementation of the "ConvMixer" architecture presented in the paper "Patches Are All You Need?"
Checklist
Ensure your pull request meets the following requirements:
cross-entropy-loss.ipynb
).notebooks/
.