[Fix] Fix pre-commit-hook for python3.12

Author: HAOCHENYE

.github/workflows/lint.yml

@@ -11,12 +11,13 @@ jobs:
     runs-on: ubuntu-22.04
     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 setuptools
           pip install pre-commit
           pre-commit install
       - name: Linting

.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
@@ -53,15 +57,16 @@ repos:
     hooks:
       - id: check-copyright
         args: ["mmcv", "tests", "--excludes", "mmcv/ops"]
-  - repo: https://gitee.com/openmmlab/mirrors-mypy
-    rev: v0.812
+  - repo: https://github.com/pre-commit/mirrors-mypy
+    rev: v1.2.0
     hooks:
       - id: mypy
         exclude: |-
           (?x)(
               ^test
               | ^docs
           )
+        additional_dependencies: ["types-setuptools", "types-requests"]
   # - repo: local
   #   hooks:
   #     - id: clang-format

.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
@@ -39,7 +43,7 @@ repos:
           - mdformat_frontmatter
           - linkify-it-py
   - repo: https://github.com/myint/docformatter
-    rev: v1.3.1
+    rev: 06907d0
     hooks:
       - id: docformatter
         args: ["--in-place", "--wrap-descriptions", "79"]
@@ -54,14 +58,15 @@ repos:
       - id: check-copyright
         args: ["mmcv", "tests", "--excludes", "mmcv/ops"]
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v0.812
+    rev: v1.2.0
     hooks:
       - id: mypy
         exclude: |-
           (?x)(
               ^test
               | ^docs
           )
+        additional_dependencies: ["types-setuptools", "types-requests"]
   # - repo: local
   #   hooks:
   #     - id: clang-format

mmcv/cnn/bricks/drop.py

@@ -13,7 +13,8 @@ def drop_path(x: torch.Tensor,
     residual blocks).
 
     We follow the implementation
-    https://github.com/rwightman/pytorch-image-models/blob/a2727c1bf78ba0d7b5727f5f95e37fb7f8866b1f/timm/models/layers/drop.py  # noqa: E501
+    https://github.com/rwightman/pytorch-image-models/blob/a2727c1bf78ba0d7b5727f5f95e37fb7f8866b1f/timm/models/layers/drop.py
+    # noqa: E501
     """
     if drop_prob == 0. or not training:
         return x

mmcv/cnn/rfsearch/search.py

@@ -71,7 +71,7 @@ def __init__(self,
         self.by_epoch = by_epoch
 
     def init_model(self, model: nn.Module):
-        """init model with search ability.
+        """Init model with search ability.
 
         Args:
             model (nn.Module): pytorch model
@@ -132,7 +132,7 @@ def step(self, model: nn.Module, work_dir: str) -> None:
             )
 
     def estimate_and_expand(self, model: nn.Module) -> None:
-        """estimate and search for RFConvOp.
+        """Estimate and search for RFConvOp.
 
         Args:
             model (nn.Module): pytorch model
@@ -146,7 +146,7 @@ def wrap_model(self,
                    model: nn.Module,
                    search_op: str = 'Conv2d',
                    prefix: str = '') -> None:
-        """wrap model to support searchable conv op.
+        """Wrap model to support searchable conv op.
 
         Args:
             model (nn.Module): pytorch model
@@ -187,7 +187,7 @@ def set_model(self,
                   search_op: str = 'Conv2d',
                   init_rates: Optional[int] = None,
                   prefix: str = '') -> None:
-        """set model based on config.
+        """Set model based on config.
 
         Args:
             model (nn.Module): pytorch model

mmcv/cnn/rfsearch/utils.py

@@ -4,7 +4,7 @@
 
 
 def write_to_json(config: dict, filename: str):
-    """save config to json file.
+    """Save config to json file.
 
     Args:
         config (dict): Config to be saved.
@@ -16,7 +16,7 @@ def write_to_json(config: dict, filename: str):
 
 
 def expand_rates(dilation: tuple, config: dict) -> list:
-    """expand dilation rate according to config.
+    """Expand dilation rate according to config.
 
     Args:
         dilation (int): _description_

mmcv/ops/active_rotated_filter.py

@@ -16,9 +16,9 @@ class ActiveRotatedFilterFunction(Function):
     """Encoding the orientation information and generating orientation-
     sensitive features.
 
-    The details are described in the paper `Align Deep Features for Oriented
-    Object Detection  <https://arxiv.org/abs/2008.09397>_`.
-    """
+    The details are described in the paper
+    `Align Deep Features for Oriented Object Detection  <https://arxiv.org/abs/2008.09397>_`.
+    """  # noqa: E501
 
     @staticmethod
     def forward(ctx, input: torch.Tensor,

mmcv/ops/cc_attention.py

@@ -51,7 +51,7 @@ def __init__(self, in_channels: int) -> None:
         self.in_channels = in_channels
 
     def forward(self, x: torch.Tensor) -> torch.Tensor:
-        """forward function of Criss-Cross Attention.
+        """Forward function of Criss-Cross Attention.
 
         Args:
             x (torch.Tensor): Input feature with the shape of

mmcv/ops/chamfer_distance.py

@@ -16,7 +16,8 @@ class ChamferDistanceFunction(Function):
     """This is an implementation of the 2D Chamfer Distance.
 
     It has been used in the paper `Oriented RepPoints for Aerial Object
-    Detection (CVPR 2022) <https://arxiv.org/abs/2105.11111>_`.
+    Detection (CVPR 2022)
+    <https://arxiv.org/abs/2105.11111>_`.
     """
 
     @staticmethod

mmcv/ops/correlation.py

@@ -116,7 +116,7 @@ def _output_size(ctx, input1):
 
 
 class Correlation(nn.Module):
-    r"""Correlation operator
+    r"""Correlation operator.
 
     This correlation operator works for optical flow correlation computation.