Move param handlers to core part
See original GitHub issue🚀 Feature
As discussed in https://github.com/Project-MONAI/MONAI/pull/1949#issuecomment-815283287 moving https://github.com/pytorch/ignite/blob/master/ignite/contrib/handlers/param_scheduler.py to the core part: https://github.com/pytorch/ignite/tree/master/ignite/handlers could be great addition for users, apps and toolkits which do not use contrib module (e.g. MONAI).
While moving the code, we would like to ensure BC-compatibilty for at least one future release. Once the code has been moved we can put deprecation message for classes and methods inside contrib module. contrib/handlers/param_scheduler.py
will be removed in v0.6.0.
A follow-up feature to this feature request could be enlarging the scope of ParamScheduler
to scheduling any parameter (e.g. hyperparameters) like it is done in MONAI: https://github.com/Project-MONAI/MONAI/pull/1949
Issue Analytics
- State:
- Created 2 years ago
- Comments:8 (1 by maintainers)
@vfdev-5 perfect, will aim to have it done by the end of the weekend.
@vfdev-5 thanks for ccing.
Just to make sure I get your right.
For now we will just move ParamScheduler out of contrib and add MONAI ParameterScheduler to Ignite?
In this issue we will not aim to combine the two into a better class because we want to maintain backwards compatibility between 0.5.1 and 0.5. Right?
Thanks!