-
Notifications
You must be signed in to change notification settings - Fork 8
Meta cat upd #21
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
Meta cat upd #21
Conversation
Improved by addressing: - saving model by overwriting the existing one as default - mentioning the function that auto adjusts for variations in task and class names - Added note that training from scratch is not recommended
@shubham-s-agarwal perhaps you want to sync with |
This still has merge conflicts. Please sync / merge |
Done! |
You also want to make the workflow run on a later runner. E.g Though technically it's a separate issue and would better fit in a separate PR. But we've not been that pedantic on WWC. |
I.e the AdamW import that was fixed in #523 (released in 1.16).
Made some changes:
So now workflow runs and is successful. @shubham-s-agarwal |
Looks good to me! |
Improved documentation by addressing: