-
Notifications
You must be signed in to change notification settings - Fork 27.4k
[FR] rename LR scheduler base class _LRScheduler to be non-private #61232
Copy link
Copy link
Closed
Labels
actionableenhancementNot as big of a feature, but technically not a bug. Should be easy to fixNot as big of a feature, but technically not a bug. Should be easy to fixhigh prioritymodule: optimizerRelated to torch.optimRelated to torch.optimtriagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate moduleThis issue has been looked at a team member, and triaged and prioritized into an appropriate module
Description
🚀 Feature
Rename _LRScheduler to something like LRScheduler without underscore.
Motivation
Allows for writing typed code much better. We have torch.optim.Optimizer but no public base class for learning rate schedulers.
cc @ezyang @gchanan @zou3519 @vincentqb @jbschlosser @albanD @iramazanli
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
actionableenhancementNot as big of a feature, but technically not a bug. Should be easy to fixNot as big of a feature, but technically not a bug. Should be easy to fixhigh prioritymodule: optimizerRelated to torch.optimRelated to torch.optimtriagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate moduleThis issue has been looked at a team member, and triaged and prioritized into an appropriate module
Type
Projects
Status
Done