Rename local vars

Author: akihironitta

pytorch_lightning/core/optimizer.py

@@ -184,14 +184,14 @@ def _init_optimizers_and_lr_schedulers(
         optim_conf = _MockOptimizer()
 
     optimizers, lr_schedulers, optimizer_frequencies, monitor = _configure_optimizers(optim_conf)
-    lr_schedulers = (
+    lr_scheduler_configs = (
         _configure_schedulers_automatic_opt(lr_schedulers, monitor)
         if model.automatic_optimization
         else _configure_schedulers_manual_opt(lr_schedulers)
     )
-    _set_scheduler_opt_idx(optimizers, lr_schedulers)
-    _validate_scheduler_api(lr_schedulers, model)
-    return optimizers, lr_schedulers, optimizer_frequencies
+    _set_scheduler_opt_idx(optimizers, lr_scheduler_configs)
+    _validate_scheduler_api(lr_scheduler_configs, model)
+    return optimizers, lr_scheduler_configs, optimizer_frequencies
 
 
 def _configure_optimizers(
@@ -259,8 +259,9 @@ def _configure_optimizers(
 
 
 def _configure_schedulers_automatic_opt(schedulers: list, monitor: Optional[str]) -> List[LRSchedulerConfig]:
-    """Convert each scheduler into dict structure with relevant information, when using automatic optimization."""
-    lr_schedulers = []
+    """Convert each scheduler into `LRSchedulerConfig` with relevant information, when using automatic
+    optimization."""
+    lr_scheduler_configs = []
     for scheduler in schedulers:
         if isinstance(scheduler, dict):
             # check provided keys
@@ -296,24 +297,25 @@ def _configure_schedulers_automatic_opt(schedulers: list, monitor: Optional[str]
                     " Are you sure you didn't mean 'interval': 'step'?",
                     category=RuntimeWarning,
                 )
-            scheduler = LRSchedulerConfig(**scheduler)
+            config = LRSchedulerConfig(**scheduler)
         elif isinstance(scheduler, ReduceLROnPlateau):
             if monitor is None:
                 raise MisconfigurationException(
                     "`configure_optimizers` must include a monitor when a `ReduceLROnPlateau`"
                     " scheduler is used. For example:"
                     ' {"optimizer": optimizer, "lr_scheduler": scheduler, "monitor": "metric_to_track"}'
                 )
-            scheduler = LRSchedulerConfig(scheduler, reduce_on_plateau=True, monitor=monitor)
+            config = LRSchedulerConfig(scheduler, reduce_on_plateau=True, monitor=monitor)
         else:
-            scheduler = LRSchedulerConfig(scheduler)
-        lr_schedulers.append(scheduler)
-    return lr_schedulers
+            config = LRSchedulerConfig(scheduler)
+        lr_scheduler_configs.append(config)
+    return lr_scheduler_configs
 
 
 def _configure_schedulers_manual_opt(schedulers: list) -> List[LRSchedulerConfig]:
-    """Convert each scheduler into dict structure with relevant information, when using manual optimization."""
-    lr_schedulers = []
+    """Convert each scheduler into `LRSchedulerConfig` structure with relevant information, when using manual
+    optimization."""
+    lr_scheduler_configs = []
     for scheduler in schedulers:
         if isinstance(scheduler, dict):
             invalid_keys = {"interval", "frequency", "reduce_on_plateau", "monitor", "strict"}
@@ -326,15 +328,15 @@ def _configure_schedulers_manual_opt(schedulers: list) -> List[LRSchedulerConfig
                     category=RuntimeWarning,
                 )
 
-            scheduler = LRSchedulerConfig(**{key: scheduler[key] for key in scheduler if key not in invalid_keys})
+            config = LRSchedulerConfig(**{key: scheduler[key] for key in scheduler if key not in invalid_keys})
         else:
-            scheduler = LRSchedulerConfig(scheduler)
-        lr_schedulers.append(scheduler)
-    return lr_schedulers
+            config = LRSchedulerConfig(scheduler)
+        lr_scheduler_configs.append(config)
+    return lr_scheduler_configs
 
 
-def _validate_scheduler_api(lr_schedulers: List[LRSchedulerConfig], model: "pl.LightningModule") -> None:
-    for config in lr_schedulers:
+def _validate_scheduler_api(lr_scheduler_configs: List[LRSchedulerConfig], model: "pl.LightningModule") -> None:
+    for config in lr_scheduler_configs:
         scheduler = config.scheduler
         if not isinstance(scheduler, _SupportsStateDict):
             raise TypeError(