Remove the broken and unused get_deprecated_arg_names

Author: carmocca

src/pytorch_lightning/core/datamodule.py

@@ -113,15 +113,6 @@ def get_init_arguments_and_types(cls) -> List[Tuple[str, Tuple, Any]]:
         """
         return get_init_arguments_and_types(cls)
 
-    @classmethod
-    def get_deprecated_arg_names(cls) -> List:
-        """Returns a list with deprecated DataModule arguments."""
-        depr_arg_names: List[str] = []
-        for name, val in cls.__dict__.items():
-            if name.startswith("DEPRECATED") and isinstance(val, (tuple, list)):
-                depr_arg_names.extend(val)
-        return depr_arg_names
-
     @classmethod
     def from_datasets(
         cls,

src/pytorch_lightning/trainer/trainer.py

@@ -2434,15 +2434,6 @@ def default_attributes(cls) -> dict:
         init_signature = inspect.signature(cls)
         return {k: v.default for k, v in init_signature.parameters.items()}
 
-    @classmethod
-    def get_deprecated_arg_names(cls) -> List:
-        """Returns a list with deprecated Trainer arguments."""
-        depr_arg_names = []
-        for name, val in cls.__dict__.items():
-            if name.startswith("DEPRECATED") and isinstance(val, (tuple, list)):
-                depr_arg_names.extend(val)
-        return depr_arg_names
-
     @classmethod
     def from_argparse_args(cls: Any, args: Union[Namespace, ArgumentParser], **kwargs) -> Any:
         return from_argparse_args(cls, args, **kwargs)

src/pytorch_lightning/utilities/argparse.py

@@ -212,8 +212,6 @@ def add_argparse_args(
         parser = ArgumentParser(parents=[parent_parser], add_help=False)
 
     ignore_arg_names = ["self", "args", "kwargs"]
-    if hasattr(cls, "get_deprecated_arg_names"):
-        ignore_arg_names += cls.get_deprecated_arg_names()
 
     allowed_types = (str, int, float, bool)
 

tests/tests_pytorch/utilities/test_argparse.py

@@ -1,6 +1,6 @@
 import io
 from argparse import ArgumentParser, Namespace
-from typing import Generic, List, TypeVar
+from typing import Generic, TypeVar
 from unittest.mock import MagicMock
 
 import pytest
@@ -118,10 +118,6 @@ class AddArgparseArgsExampleClass:
     def __init__(self, my_parameter: int = 0):
         pass
 
-    @staticmethod
-    def get_deprecated_arg_names() -> List[str]:
-        return []
-
 
 class AddArgparseArgsExampleClassViaInit:
     def __init__(self, my_parameter: int = 0):