Dropper TrainsLoger and TrainsSaver also removed the backward compati… (#2742)

* Dropper TrainsLoger and TrainsSaver also removed the backward compatibility

* Update clearml_logger.py

* Update handlers.rst

* Replaced RuntimeError with ModuleNotFoundError

* Update ignite/contrib/handlers/clearml_logger.py

Co-authored-by: Somasree <[email protected]>
Co-authored-by: vfdev <[email protected]>

Author: 3 people

docs/source/contrib/handlers.rst

@@ -42,7 +42,7 @@ Loggers
     polyaxon_logger
     tensorboard_logger
     tqdm_logger
-    trains_logger
+   
     visdom_logger
     wandb_logger
 

ignite/contrib/handlers/__init__.py

@@ -4,7 +4,7 @@
 from ignite.contrib.handlers.polyaxon_logger import PolyaxonLogger
 from ignite.contrib.handlers.tensorboard_logger import TensorboardLogger
 from ignite.contrib.handlers.tqdm_logger import ProgressBar
-from ignite.contrib.handlers.trains_logger import TrainsLogger
+
 from ignite.contrib.handlers.visdom_logger import VisdomLogger
 from ignite.contrib.handlers.wandb_logger import WandBLogger
 from ignite.handlers import EpochOutputStore, global_step_from_engine  # ref  # ref

ignite/contrib/handlers/clearml_logger.py

@@ -115,15 +115,10 @@ def __init__(self, **kwargs: Any):
             from clearml import Task
             from clearml.binding.frameworks.tensorflow_bind import WeightsGradientHistHelper
         except ImportError:
-            try:
-                # Backwards-compatibility for legacy Trains SDK
-                from trains import Task
-                from trains.binding.frameworks.tensorflow_bind import WeightsGradientHistHelper
-            except ImportError:
-                raise RuntimeError(
-                    "This contrib module requires clearml to be installed. "
-                    "You may install clearml using: \n pip install clearml \n"
-                )
+            raise RuntimeError(
+                "This contrib module requires clearml to be installed. "
+                "You may install clearml using: \n pip install clearml \n"
+            )
 
         experiment_kwargs = {k: v for k, v in kwargs.items() if k not in ("project_name", "task_name", "task_type")}
 

ignite/contrib/handlers/trains_logger.py

@@ -21,18 +21,11 @@
     WeightsHistHandler,
     WeightsScalarHandler,
 )
-from ignite.contrib.handlers.trains_logger import TrainsLogger
+
 from ignite.engine import Engine, Events, State
 from ignite.handlers import Checkpoint
 
 
-def test_trains_logger_alias(dirname):
-    with pytest.warns(UserWarning, match="ClearMLSaver: running in bypass mode"):
-        TrainsLogger.set_bypass_mode(True)
-        logger = TrainsLogger(output_uri=dirname)
-        assert isinstance(logger, ClearMLLogger)
-
-
 def test_optimizer_params_handler_wrong_setup():
 
     with pytest.raises(TypeError):