Skip to content

Commit 8c01e82

Browse files
NeoKishrohitgr7awaelchli
authored
Remove deprecated trainer_optimizer_mixin (#14887)
Co-authored-by: Rohit Gupta <[email protected]> Co-authored-by: Adrian Wälchli <[email protected]>
1 parent dce5644 commit 8c01e82

File tree

4 files changed

+7
-89
lines changed

4 files changed

+7
-89
lines changed

src/pytorch_lightning/CHANGELOG.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
222222
- Removed the deprecated `Trainer.use_amp` and `LightningModule.use_amp` attributes ([#14832](https://github.com/Lightning-AI/lightning/pull/14832))
223223

224224

225-
- Removed the deprecated `Trainer.run_stage` in favor of `Trainer.{fit,validate,test,predict}`
225+
- Removed the deprecated `Trainer.run_stage` in favor of `Trainer.{fit,validate,test,predict}` ([#14870](https://github.com/Lightning-AI/lightning/pull/14870))
226226

227227

228228
- Removed the deprecated `SimpleProfiler.profile_iterable` and `AdvancedProfiler.profile_iterable` attributes ([#14864](https://github.com/Lightning-AI/lightning/pull/14864))
@@ -231,7 +231,11 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
231231
- Removed the deprecated `Trainer.verbose_evaluate` ([#14884](https://github.com/Lightning-AI/lightning/pull/14884))
232232

233233

234-
- Remove the deprecated `Trainer.should_rank_save_checkpoint` ([#14885](https://github.com/Lightning-AI/lightning/pull/14885))
234+
- Removed the deprecated `Trainer.should_rank_save_checkpoint` ([#14885](https://github.com/Lightning-AI/lightning/pull/14885))
235+
236+
237+
- Removed the deprecated `TrainerOptimizersMixin` ([#14887](https://github.com/Lightning-AI/lightning/pull/14887))
238+
235239

236240
- Removed the deprecated `Trainer.lightning_optimizers` ([#14889](https://github.com/Lightning-AI/lightning/pull/14889))
237241

src/pytorch_lightning/trainer/optimizers.py

Lines changed: 0 additions & 65 deletions
This file was deleted.

src/pytorch_lightning/trainer/trainer.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,6 @@
7474
from pytorch_lightning.trainer.connectors.logger_connector import LoggerConnector
7575
from pytorch_lightning.trainer.connectors.logger_connector.result import _ResultCollection
7676
from pytorch_lightning.trainer.connectors.signal_connector import SignalConnector
77-
from pytorch_lightning.trainer.optimizers import TrainerOptimizersMixin
7877
from pytorch_lightning.trainer.states import RunningStage, TrainerFn, TrainerState, TrainerStatus
7978
from pytorch_lightning.trainer.supporters import CombinedLoader
8079
from pytorch_lightning.tuner.tuning import _TunerResult, Tuner
@@ -108,9 +107,7 @@
108107
)
109108

110109

111-
class Trainer(
112-
TrainerOptimizersMixin, # TODO: Remove in v1.8
113-
):
110+
class Trainer:
114111
@_defaults_from_env_vars
115112
def __init__(
116113
self,

tests/tests_pytorch/deprecated_api/test_remove_1-8.py

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -62,24 +62,6 @@ def test_v1_8_0_trainer_ckpt_path_attributes(fn_prefix: str):
6262
setattr(trainer, test_attr, "v")
6363

6464

65-
def test_v1_8_0_trainer_optimizers_mixin():
66-
trainer = Trainer()
67-
model = BoringModel()
68-
trainer.strategy.connect(model)
69-
trainer.lightning_module.trainer = trainer
70-
71-
with pytest.deprecated_call(
72-
match=r"`TrainerOptimizersMixin.init_optimizers` was deprecated in v1.6 and will be removed in v1.8."
73-
):
74-
trainer.init_optimizers(model)
75-
76-
with pytest.deprecated_call(
77-
match=r"`TrainerOptimizersMixin.convert_to_lightning_optimizers` was deprecated in v1.6 and will be removed in "
78-
"v1.8."
79-
):
80-
trainer.convert_to_lightning_optimizers()
81-
82-
8365
def test_v_1_8_0_deprecated_device_stats_monitor_prefix_metric_keys():
8466
from pytorch_lightning.callbacks.device_stats_monitor import prefix_metric_keys
8567

0 commit comments

Comments
 (0)