-
Notifications
You must be signed in to change notification settings - Fork 332
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
Global rename of classes
to num_classes
#1474
Conversation
/gcbrun |
/gcbrun |
#657 👀 -- we should make sure we have strong consensus before merging this PR, since apparently we've changed our mind's before, and this is only going to get harder / more intrusive for users as time goes on. I'll make a PR to KerasIO to update this as well (which we can wait to merge until we cut a release here) |
Corresponding KerasIO change: keras-team/keras-io#1271 |
Amazing @ianstenbit thank you 🚀 🚀 🚀 |
* Global rename of classes to num_classes * fix training scripts * Fix some box keys * Fix RetinaNet train * Fix more box keys; * Fix more box keys * Fix more box keys * Fix demo utils * Manual layer fixes * Manual losses + metrics fixes * Fix number of num_ issue * Manual models + utils fixes * Formatting * Minor rcnn fix * Manual bbox fixes * Manual misc fixes * Manual misc fixes * Fix convmixer
Fixes #1473