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
OK, I actually see that ReduceLROnPlateau.step requires a metrics argument, while other PyTorch LR schedulers do not. This which would complicate Tango's training implementation, so the fix is not as easy as I thought.
🐛 Describe the bug
For some reason, PyTorch's
ReduceLROnPlateau
learning rate scheduler is not registered as a subclass ofLRScheduler
. This causes Tango's registration to fail to register it, as Tango uses a subclass ofLRScheduler
check to see whether a given class ought to be registered.Suggested fix: manually include
ReduceLROnPlateau
in Tango's registration code. I'd be happy to open a PR for this.Versions
The text was updated successfully, but these errors were encountered: