Delete deprecated `TrainerLoggingMixin` (#8609)

* Delete deprecated `TrainerLoggingMixin`
* Update CHANGELOG
* Delete from Trainer

CHANGELOG.md

@@ -58,10 +58,10 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
 - Removed deprecated `metrics` ([#8586](https://github.com/PyTorchLightning/pytorch-lightning/pull/8586/))
 
 
-- Removed the `outputs` argument in both the `LightningModule.on_train_epoch_end` and `Callback.on_train_epoch_end` hooks ([#8587](https://github.com/PyTorchLightning/pytorch-lightning/pull/8587))
+- Removed the deprecated `outputs` argument in both the `LightningModule.on_train_epoch_end` and `Callback.on_train_epoch_end` hooks ([#8587](https://github.com/PyTorchLightning/pytorch-lightning/pull/8587))
 
 
--
+- Delete the deprecated `TrainerLoggingMixin` class ([#8609](https://github.com/PyTorchLightning/pytorch-lightning/pull/8609))
 
 
 -

pytorch_lightning/trainer/logging.py

@@ -1,34 +0,0 @@
-# Copyright The PyTorch Lightning team.
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-from abc import ABC
-
-from pytorch_lightning.utilities import rank_zero_deprecation
-from pytorch_lightning.utilities.metrics import metrics_to_scalars as new_metrics_to_scalars
-
-
-class TrainerLoggingMixin(ABC):
-    """
-    TODO: Remove this class in v1.5.
-
-    Use the utilities from ``pytorch_lightning.utilities.metrics`` instead.
-    """
-
-    def metrics_to_scalars(self, metrics: dict) -> dict:
-        rank_zero_deprecation(
-            "Internal: TrainerLoggingMixin.metrics_to_scalars is deprecated in v1.3"
-            " and will be removed in v1.5."
-            " Use `pytorch_lightning.utilities.metrics.metrics_to_scalars` instead."
-        )
-        return new_metrics_to_scalars(metrics)

pytorch_lightning/trainer/trainer.py

@@ -58,7 +58,6 @@ from pytorch_lightning.trainer.connectors.slurm_connector import SLURMConnector
 from pytorch_lightning.trainer.connectors.training_trick_connector import TrainingTricksConnector
 from pytorch_lightning.trainer.data_loading import TrainerDataLoadingMixin
 from pytorch_lightning.trainer.deprecated_api import DeprecatedTrainerAttributes
-from pytorch_lightning.trainer.logging import TrainerLoggingMixin
 from pytorch_lightning.trainer.model_hooks import TrainerModelHooksMixin
 from pytorch_lightning.trainer.optimizers import TrainerOptimizersMixin
 from pytorch_lightning.trainer.properties import TrainerProperties
@@ -97,7 +96,6 @@ class Trainer(
     TrainerCallbackHookMixin,
     TrainerModelHooksMixin,
     TrainerOptimizersMixin,
-    TrainerLoggingMixin,
     TrainerTrainingTricksMixin,
     TrainerDataLoadingMixin,
     DeprecatedTrainerAttributes,

tests/deprecated_api/test_remove_1-5.py

@@ -247,12 +247,6 @@ def test_v1_5_0_lightning_module_write_prediction(tmpdir):
         trainer.test(model)
 
 
-def test_v1_5_0_trainer_logging_mixin(tmpdir):
-    trainer = Trainer(default_root_dir=tmpdir, max_epochs=1, checkpoint_callback=False, logger=False)
-    with pytest.deprecated_call(match="is deprecated in v1.3 and will be removed in v1.5"):
-        trainer.metrics_to_scalars({})
-
-
 def test_v1_5_0_lighting_module_grad_norm(tmpdir):
     model = BoringModel()
     with pytest.deprecated_call(match="is deprecated in v1.3 and will be removed in v1.5"):