-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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 Trainer states #1633
Comments
Hi! thanks for your contribution!, great first issue! |
As I understood it right, the change is to add enum with phases, add corresponding updates, and logging of the current phase. @Borda I can take it as my first issue. |
@zerogerc cool, that it adds an enum (with possible texts) and change the states according to the training phase... |
Don't we want to control these states with decorators? I think having the code for these state changes mixed with the core training logic is a bit distracting and fragile, don't you think? |
well, decorators would be nicer :] @awaelchli mind comment/suggest on the open PR? |
🚀 Feature
Add trainer states so in each time user get info what is happening
Motivation
simplify the return values and clean up the interface
Pitch
The state will be implemented as enum and updated according to Trainer phase:
The text was updated successfully, but these errors were encountered: