upgrade ruff to 0.11.13 (#5003)

Author: leoll2

.pre-commit-config.yaml

@@ -69,8 +69,8 @@ repos:
 
   # Ruff
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.4.10
+    rev: v0.11.13
     hooks:
-      - id: ruff
-        args: [--fix]
+      - id: ruff-check
+        args: [--fix, --unsafe-fixes]
       - id: ruff-format

application/backend/pyproject.toml

@@ -51,7 +51,7 @@ dev = [
     "time-machine~=2.19.0",
 ]
 lint = [
-    "ruff==0.9.3",
+    "ruff==0.11.13",
     "mypy~=1.16.1",
     "types-PyYAML~=6.0.12",
     "types-requests~=2.32.4",

application/backend/uv.lock

@@ -291,19 +291,17 @@ ignore = [
     "D105",     # Missing docstring in magic method
 
     # flake8-annotations
-    "ANN101",   # Missing-type-self
     "ANN002",   # Missing type annotation for *args
     "ANN003",   # Missing type annotation for **kwargs
 
-    "ANN102",   # Missing type annotation for `cls` in classmethod
     "ANN204",   # Missing return type annotation for special method `__init__`
 
     "ARG002",   # Unused method argument -> some function cannot use argument
 
     "COM812",   # Missing trailing comma -> conflicts with ISC001
 
     # flake8-type-checking
-    "TCH001",   # typing-only-first-party-import, Sometimes this causes an incorrect error.
+    "TC001",   # typing-only-first-party-import, Sometimes this causes an incorrect error.
     # flake8-fixme
     "FIX002",   # line-contains-todo
 
@@ -342,15 +340,15 @@ exclude = [
     "venv",
     "tests/assets",
 
-    # it will be cleaned up later
-    "for_developers/helpers.py",
-
     # Ruff complains it but don't know how to fix since it literally showed no useful logs.
     # https://github.com/openvinotoolkit/training_extensions/actions/runs/7176557723/job/19541622452?pr=2718#step:5:170
     "tests/regression/*.py",
 
     # Mostly borrowed from jsonargparse codebase
-    "src/otx/core/utils/jsonargparse.py"
+    "src/otx/core/utils/jsonargparse.py",
+
+    # Sphinx auto-generated files
+    "docs/source/conf.py",
 ]
 
 # Allow unused variables when underscore-prefixed.
@@ -391,6 +389,7 @@ notice-rgx = """
 # Declare an additional exclude rule for test code
 "tests/**/*.py" = [
     "S101",     # pytest-style allows `assert` statements in tests.
+    "S603",     # Allow for untrusted input in tests.
     "SLF001",   # We sometimes need to inspect private functions for testing.
     "TCH003",   # It doesn't seem necessary to use TYPE_CHECKING in tests.
     "PT004",    # fixture ignore type returning.

library/pyproject.toml

@@ -8,4 +8,4 @@
 
 from .utils import get_otx_root_path, list_models
 
-__all__ = ["list_models", "get_otx_root_path"]
+__all__ = ["get_otx_root_path", "list_models"]

library/src/otx/backend/native/cli/__init__.py

@@ -129,8 +129,7 @@ def __init__(
             input_size = self._datamodule.input_size
             if input_size is None:
                 msg = (
-                    "Input size is not specified in the datamodule. "
-                    "Ensure that the datamodule has a valid input size."
+                    "Input size is not specified in the datamodule. Ensure that the datamodule has a valid input size."
                 )
                 raise ValueError(msg)
             get_model_args["data_input_params"] = DataInputParams(

library/src/otx/backend/native/engine.py

@@ -237,11 +237,11 @@ def to_exportable_code(
     @staticmethod
     def _embed_onnx_metadata(onnx_model: onnx.ModelProto, metadata: dict[tuple[str, str], Any]) -> onnx.ModelProto:
         """Embeds metadata to ONNX model."""
-        for item in metadata:
+        for k, v in metadata.items():
             meta = onnx_model.metadata_props.add()
-            attr_path = " ".join(map(str, item))
+            attr_path = " ".join(map(str, k))
             meta.key = attr_path.strip()
-            meta.value = str(metadata[item])
+            meta.value = str(v)
 
         return onnx_model
 

library/src/otx/backend/native/exporter/base.py

@@ -15,13 +15,13 @@
 )
 
 __all__ = [
-    "SyncExecutor",
     "AsyncExecutor",
-    "create_visualizer",
-    "ModelWrapper",
     "BaseVisualizer",
     "ClassificationVisualizer",
-    "SemanticSegmentationVisualizer",
     "InstanceSegmentationVisualizer",
+    "ModelWrapper",
     "ObjectDetectionVisualizer",
+    "SemanticSegmentationVisualizer",
+    "SyncExecutor",
+    "create_visualizer",
 ]

library/src/otx/backend/native/exporter/exportable_code/demo/demo_package/__init__.py

@@ -7,6 +7,6 @@
 from .synchronous import SyncExecutor
 
 __all__ = [
-    "SyncExecutor",
     "AsyncExecutor",
+    "SyncExecutor",
 ]

library/src/otx/backend/native/exporter/exportable_code/demo/demo_package/executors/__init__.py

@@ -14,11 +14,11 @@
 )
 
 __all__ = [
+    "BaseStreamer",
     "CameraStreamer",
     "DirStreamer",
     "ImageStreamer",
     "ThreadedStreamer",
     "VideoStreamer",
-    "BaseStreamer",
     "get_streamer",
 ]