You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hopefully not a controversial suggestion? I think training is the more natural place for LR schedulers (unless there is some dependency issue I'm missing).
The text was updated successfully, but these errors were encountered:
It seem to me that this affects on recipes, docs and 1 test. Will open PR now. I don't sure, should we save it in modules for few versions due to backward capability reasons
Thanks for creating the PR! I agree, let's keep the one in modules for now (but point everything to the version in training). Then add the deprecated flag saying that it will be removed in the 0.4 release and to use the one in training/ instead (example usage)
Thanks for creating the PR! I agree, let's keep the one in modules for now (but point everything to the version in training). Then add the deprecated flag saying that it will be removed in the 0.4 release and to use the one in training/ instead (example usage)
Hopefully not a controversial suggestion? I think training is the more natural place for LR schedulers (unless there is some dependency issue I'm missing).
The text was updated successfully, but these errors were encountered: