[Fix] Fix lint (#1598)

* [Fix] Fix lint

* [Fix] Fix lint

* Update mmengine/dist/utils.py

Co-authored-by: Zaida Zhou <[email protected]>

---------

Co-authored-by: Zaida Zhou <[email protected]>

Author: HAOCHENYE

.github/workflows/lint.yml

@@ -11,10 +11,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Set up Python 3.7
+      - name: Set up Python 3.10.15
         uses: actions/setup-python@v2
         with:
-          python-version: 3.7
+          python-version: '3.10.15'
       - name: Install pre-commit hook
         run: |
           pip install pre-commit

.github/workflows/pr_stage_test.yml

@@ -1,5 +1,9 @@
 name: pr_stage_test
 
+env:
+  ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true
+
+
 on:
   pull_request:
     paths-ignore:

.pre-commit-config-zh-cn.yaml

@@ -1,7 +1,11 @@
 exclude: ^tests/data/
 repos:
-  - repo: https://gitee.com/openmmlab/mirrors-flake8
-    rev: 5.0.4
+  - repo: https://github.com/pre-commit/pre-commit
+    rev: v4.0.0
+    hooks:
+      - id: validate_manifest
+  - repo: https://github.com/PyCQA/flake8
+    rev: 7.1.1
     hooks:
       - id: flake8
   - repo: https://gitee.com/openmmlab/mirrors-isort
@@ -13,7 +17,7 @@ repos:
     hooks:
       - id: yapf
   - repo: https://gitee.com/openmmlab/mirrors-pre-commit-hooks
-    rev: v4.3.0
+    rev: v5.0.0
     hooks:
       - id: trailing-whitespace
       - id: check-yaml
@@ -55,11 +59,12 @@ repos:
         args: ["mmengine", "tests"]
       - id: remove-improper-eol-in-cn-docs
   - repo: https://gitee.com/openmmlab/mirrors-mypy
-    rev: v0.812
+    rev: v1.2.0
     hooks:
       - id: mypy
         exclude: |-
           (?x)(
               ^examples
               | ^docs
           )
+        additional_dependencies: ["types-setuptools", "types-requests", "types-PyYAML"]

.pre-commit-config.yaml

@@ -1,7 +1,11 @@
 exclude: ^tests/data/
 repos:
+  - repo: https://github.com/pre-commit/pre-commit
+    rev: v4.0.0
+    hooks:
+      - id: validate_manifest
   - repo: https://github.com/PyCQA/flake8
-    rev: 5.0.4
+    rev: 7.1.1
     hooks:
       - id: flake8
   - repo: https://github.com/PyCQA/isort
@@ -13,7 +17,7 @@ repos:
     hooks:
       - id: yapf
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+    rev: v5.0.0
     hooks:
       - id: trailing-whitespace
       - id: check-yaml
@@ -34,12 +38,8 @@ repos:
           - mdformat-openmmlab
           - mdformat_frontmatter
           - linkify-it-py
-  - repo: https://github.com/codespell-project/codespell
-    rev: v2.2.1
-    hooks:
-      - id: codespell
   - repo: https://github.com/myint/docformatter
-    rev: v1.3.1
+    rev: 06907d0
     hooks:
       - id: docformatter
         args: ["--in-place", "--wrap-descriptions", "79"]
@@ -55,11 +55,12 @@ repos:
         args: ["mmengine", "tests"]
       - id: remove-improper-eol-in-cn-docs
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v0.812
+    rev: v1.2.0
     hooks:
       - id: mypy
         exclude: |-
           (?x)(
               ^examples
               | ^docs
           )
+        additional_dependencies: ["types-setuptools", "types-requests", "types-PyYAML"]

mmengine/_strategy/base.py

@@ -499,7 +499,7 @@ def build_optim_wrapper(
                             '"type" and "constructor" are not in '
                             f'optimizer, but got {name}={optim}')
                     optim_wrappers[name] = optim
-                return OptimWrapperDict(**optim_wrappers)
+                return OptimWrapperDict(**optim_wrappers)  # type: ignore
         else:
             raise TypeError('optimizer wrapper should be an OptimWrapper '
                             f'object or dict, but got {optim_wrapper}')

mmengine/_strategy/colossalai.py

@@ -361,7 +361,7 @@ def resume(
         map_location: Union[str, Callable] = 'default',
         callback: Optional[Callable] = None,
     ) -> dict:
-        """override this method since colossalai resume optimizer from filename
+        """Override this method since colossalai resume optimizer from filename
         directly."""
         self.logger.info(f'Resume checkpoint from {filename}')
 

mmengine/_strategy/distributed.py

@@ -53,7 +53,7 @@ def _setup_distributed(  # type: ignore
             init_dist(launcher, backend, **kwargs)
 
     def convert_model(self, model: nn.Module) -> nn.Module:
-        """convert all ``BatchNorm`` layers in the model to ``SyncBatchNorm``
+        """Convert all ``BatchNorm`` layers in the model to ``SyncBatchNorm``
         (SyncBN) or ``mmcv.ops.sync_bn.SyncBatchNorm`` (MMSyncBN) layers.
 
         Args:

mmengine/config/config.py

@@ -393,7 +393,7 @@ class Config:
 
     def __init__(
         self,
-        cfg_dict: dict = None,
+        cfg_dict: Optional[dict] = None,
         cfg_text: Optional[str] = None,
         filename: Optional[Union[str, Path]] = None,
         env_variables: Optional[dict] = None,
@@ -1227,7 +1227,8 @@ def is_base_line(c):
                 if base_code is not None:
                     base_code = ast.Expression(  # type: ignore
                         body=base_code.value)  # type: ignore
-                    base_files = eval(compile(base_code, '', mode='eval'))
+                    base_files = eval(compile(base_code, '',
+                                              mode='eval'))  # type: ignore
                 else:
                     base_files = []
         elif file_format in ('.yml', '.yaml', '.json'):
@@ -1288,7 +1289,7 @@ def _get_cfg_path(cfg_path: str,
     def _merge_a_into_b(a: dict,
                         b: dict,
                         allow_list_keys: bool = False) -> dict:
-        """merge dict ``a`` into dict ``b`` (non-inplace).
+        """Merge dict ``a`` into dict ``b`` (non-inplace).
 
         Values in ``a`` will overwrite ``b``. ``b`` is copied first to avoid
         in-place modifications.
@@ -1358,22 +1359,22 @@ def auto_argparser(description=None):
 
     @property
     def filename(self) -> str:
-        """get file name of config."""
+        """Get file name of config."""
         return self._filename
 
     @property
     def text(self) -> str:
-        """get config text."""
+        """Get config text."""
         return self._text
 
     @property
     def env_variables(self) -> dict:
-        """get used environment variables."""
+        """Get used environment variables."""
         return self._env_variables
 
     @property
     def pretty_text(self) -> str:
-        """get formatted python config text."""
+        """Get formatted python config text."""
 
         indent = 4
 
@@ -1727,17 +1728,17 @@ def to_dict(self, keep_imported: bool = False):
 
 
 class DictAction(Action):
-    """
-    argparse action to split an argument into KEY=VALUE form
-    on the first = and append to a dictionary. List options can
-    be passed as comma separated values, i.e 'KEY=V1,V2,V3', or with explicit
-    brackets, i.e. 'KEY=[V1,V2,V3]'. It also support nested brackets to build
-    list/tuple values. e.g. 'KEY=[(V1,V2),(V3,V4)]'
+    """Argparse action to split an argument into KEY=VALUE form on the first =
+    and append to a dictionary.
+
+    List options can be passed as comma separated values, i.e 'KEY=V1,V2,V3',
+    or with explicit brackets, i.e. 'KEY=[V1,V2,V3]'. It also support nested
+    brackets to build list/tuple values. e.g. 'KEY=[(V1,V2),(V3,V4)]'
     """
 
     @staticmethod
     def _parse_int_float_bool(val: str) -> Union[int, float, bool, Any]:
-        """parse int/float/bool value in the string."""
+        """Parse int/float/bool value in the string."""
         try:
             return int(val)
         except ValueError:
@@ -1822,7 +1823,7 @@ def __call__(self,
                  parser: ArgumentParser,
                  namespace: Namespace,
                  values: Union[str, Sequence[Any], None],
-                 option_string: str = None):
+                 option_string: str = None):  # type: ignore
         """Parse Variables in string and add them into argparser.
 
         Args:

mmengine/dist/utils.py

@@ -563,7 +563,7 @@ def cast_data_device(
         Tensor or list or dict: ``data`` was casted to ``device``.
     """
     if out is not None:
-        if type(data) != type(out):
+        if type(data) is not type(out):
             raise TypeError(
                 'out should be the same type with data, but got data is '
                 f'{type(data)} and out is {type(data)}')

mmengine/evaluator/metric.py

@@ -175,11 +175,11 @@ def __init__(self,
         self.out_file_path = out_file_path
 
     def process(self, data_batch: Any, predictions: Sequence[dict]) -> None:
-        """transfer tensors in predictions to CPU."""
+        """Transfer tensors in predictions to CPU."""
         self.results.extend(_to_cpu(predictions))
 
     def compute_metrics(self, results: list) -> dict:
-        """dump the prediction results to a pickle file."""
+        """Dump the prediction results to a pickle file."""
         dump(results, self.out_file_path)
         print_log(
             f'Results has been saved to {self.out_file_path}.',
@@ -188,7 +188,7 @@ def compute_metrics(self, results: list) -> dict:
 
 
 def _to_cpu(data: Any) -> Any:
-    """transfer all tensors and BaseDataElement to cpu."""
+    """Transfer all tensors and BaseDataElement to cpu."""
     if isinstance(data, (Tensor, BaseDataElement)):
         return data.to('cpu')
     elif isinstance(data, list):