-
Notifications
You must be signed in to change notification settings - Fork 117
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
Move Image classification with ConvMixer to keras-core #941
Move Image classification with ConvMixer to keras-core #941
Conversation
divyashreepathihalli
commented
Sep 22, 2023
•
edited
Loading
edited
- fixed imports
- removed tfa dependency
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main keras-team/keras-core#941 +/- ##
=======================================
Coverage 83.83% 83.83%
=======================================
Files 319 319
Lines 28879 28879
Branches 5529 5529
=======================================
Hits 24211 24211
Misses 3148 3148
Partials 1520 1520
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Thanks for the PR! Does the example work with all backends? If so we should move it out of the tensorflow directory.
I had to change data augmentation from a sequential model to a function for it to work with all the backends. I have filed an issue with a repro here - keras-team/keras#18396 |
Keras Core is becoming Keras 3, and we're switching development to the main repository! Please reopen this PR in the keras-team/keras repository. Unfortunately we aren't able to automatically transfer PRs (but we have transferred all issues). |