[SymForce] Handle pylint useless-suppression

Reviewers: william-a,brad,nathan,ryan-b Topic: sf-useless-suppression GitOrigin-RevId: 03d21936f60764723d0dbc264bff68f292d2c361
symforce-org · Mar 28, 2024 · 42c4df7 · 42c4df7
1 parent f04f311
commit 42c4df7
Show file tree

Hide file tree

Showing 47 changed files with 111 additions and 47 deletions.
diff --git a/gen/python/sym/ops/atan_camera_cal/camera_ops.py b/gen/python/sym/ops/atan_camera_cal/camera_ops.py
diff --git a/gen/python/sym/ops/atan_camera_cal/group_ops.py b/gen/python/sym/ops/atan_camera_cal/group_ops.py
diff --git a/gen/python/sym/ops/atan_camera_cal/lie_group_ops.py b/gen/python/sym/ops/atan_camera_cal/lie_group_ops.py
diff --git a/gen/python/sym/ops/double_sphere_camera_cal/camera_ops.py b/gen/python/sym/ops/double_sphere_camera_cal/camera_ops.py
diff --git a/gen/python/sym/ops/double_sphere_camera_cal/group_ops.py b/gen/python/sym/ops/double_sphere_camera_cal/group_ops.py
diff --git a/gen/python/sym/ops/double_sphere_camera_cal/lie_group_ops.py b/gen/python/sym/ops/double_sphere_camera_cal/lie_group_ops.py
diff --git a/gen/python/sym/ops/equirectangular_camera_cal/camera_ops.py b/gen/python/sym/ops/equirectangular_camera_cal/camera_ops.py
diff --git a/gen/python/sym/ops/equirectangular_camera_cal/group_ops.py b/gen/python/sym/ops/equirectangular_camera_cal/group_ops.py
diff --git a/gen/python/sym/ops/equirectangular_camera_cal/lie_group_ops.py b/gen/python/sym/ops/equirectangular_camera_cal/lie_group_ops.py
diff --git a/gen/python/sym/ops/linear_camera_cal/camera_ops.py b/gen/python/sym/ops/linear_camera_cal/camera_ops.py
diff --git a/gen/python/sym/ops/linear_camera_cal/group_ops.py b/gen/python/sym/ops/linear_camera_cal/group_ops.py
diff --git a/gen/python/sym/ops/linear_camera_cal/lie_group_ops.py b/gen/python/sym/ops/linear_camera_cal/lie_group_ops.py
diff --git a/gen/python/sym/ops/polynomial_camera_cal/camera_ops.py b/gen/python/sym/ops/polynomial_camera_cal/camera_ops.py
diff --git a/gen/python/sym/ops/polynomial_camera_cal/group_ops.py b/gen/python/sym/ops/polynomial_camera_cal/group_ops.py
diff --git a/gen/python/sym/ops/polynomial_camera_cal/lie_group_ops.py b/gen/python/sym/ops/polynomial_camera_cal/lie_group_ops.py
diff --git a/gen/python/sym/ops/pose2/group_ops.py b/gen/python/sym/ops/pose2/group_ops.py
diff --git a/gen/python/sym/ops/pose2/lie_group_ops.py b/gen/python/sym/ops/pose2/lie_group_ops.py
diff --git a/gen/python/sym/ops/pose3/group_ops.py b/gen/python/sym/ops/pose3/group_ops.py
diff --git a/gen/python/sym/ops/pose3/lie_group_ops.py b/gen/python/sym/ops/pose3/lie_group_ops.py
diff --git a/gen/python/sym/ops/rot2/group_ops.py b/gen/python/sym/ops/rot2/group_ops.py
diff --git a/gen/python/sym/ops/rot2/lie_group_ops.py b/gen/python/sym/ops/rot2/lie_group_ops.py
diff --git a/gen/python/sym/ops/rot3/group_ops.py b/gen/python/sym/ops/rot3/group_ops.py
diff --git a/gen/python/sym/ops/rot3/lie_group_ops.py b/gen/python/sym/ops/rot3/lie_group_ops.py
diff --git a/gen/python/sym/ops/spherical_camera_cal/camera_ops.py b/gen/python/sym/ops/spherical_camera_cal/camera_ops.py
diff --git a/gen/python/sym/ops/spherical_camera_cal/group_ops.py b/gen/python/sym/ops/spherical_camera_cal/group_ops.py
diff --git a/gen/python/sym/ops/spherical_camera_cal/lie_group_ops.py b/gen/python/sym/ops/spherical_camera_cal/lie_group_ops.py
diff --git a/gen/python/sym/ops/unit3/group_ops.py b/gen/python/sym/ops/unit3/group_ops.py
diff --git a/gen/python/sym/ops/unit3/lie_group_ops.py b/gen/python/sym/ops/unit3/lie_group_ops.py
diff --git a/symforce/codegen/backends/python/templates/cam_package/ops/CLASS/camera_ops.py.jinja b/symforce/codegen/backends/python/templates/cam_package/ops/CLASS/camera_ops.py.jinja
@@ -9,7 +9,7 @@ import typing as T
 
 import numpy
 
-import sym  # pylint: disable=unused-import
+import sym  # pylint: disable=useless-suppression,unused-import
 
 
 class CameraOps(object):

diff --git a/symforce/codegen/backends/python/templates/function/FUNCTION.py.jinja b/symforce/codegen/backends/python/templates/function/FUNCTION.py.jinja
@@ -4,7 +4,11 @@
  # ---------------------------------------------------------------------------- #}
 {%- import "../util/util.jinja" as util with context -%}
 
-# pylint: disable=too-many-locals,too-many-lines,too-many-statements,unused-argument,unused-import
+# pylint: disable=useless-suppression
+# pylint: disable=too-many-locals
+# pylint: disable=too-many-lines
+# pylint: disable=too-many-statements
+# pylint: disable=unused-argument
 
 import math
 import typing as T

diff --git a/symforce/codegen/backends/python/templates/ops/CLASS/group_ops.py.jinja b/symforce/codegen/backends/python/templates/ops/CLASS/group_ops.py.jinja
@@ -9,7 +9,7 @@ import typing as T
 
 import numpy
 
-import sym  # pylint: disable=unused-import
+import sym  # pylint: disable=useless-suppression,unused-import
 
 
 class GroupOps(object):

diff --git a/symforce/codegen/backends/python/templates/ops/CLASS/lie_group_ops.py.jinja b/symforce/codegen/backends/python/templates/ops/CLASS/lie_group_ops.py.jinja
@@ -9,7 +9,7 @@ import typing as T
 
 import numpy
 
-import sym  # pylint: disable=unused-import
+import sym  # pylint: disable=useless-suppression,unused-import
 
 
 class LieGroupOps(object):

diff --git a/symforce/codegen/backends/pytorch/templates/function/FUNCTION.py.jinja b/symforce/codegen/backends/pytorch/templates/function/FUNCTION.py.jinja
@@ -4,7 +4,11 @@
  # ---------------------------------------------------------------------------- #}
 {%- import "../util/util.jinja" as util with context -%}
 
-# pylint: disable=too-many-locals,too-many-lines,too-many-statements,unused-argument
+# pylint: disable=useless-suppression
+# pylint: disable=too-many-locals
+# pylint: disable=too-many-lines
+# pylint: disable=too-many-statements
+# pylint: disable=unused-argument
 
 import math # pylint: disable=unused-import
 import typing as T

diff --git a/test/symforce_function_codegen_test_data/symengine/az_el_from_point.py b/test/symforce_function_codegen_test_data/symengine/az_el_from_point.py
@@ -4,7 +4,11 @@
 # Do NOT modify by hand.
 # -----------------------------------------------------------------------------
 
-# pylint: disable=too-many-locals,too-many-lines,too-many-statements,unused-argument,unused-import
+# pylint: disable=useless-suppression
+# pylint: disable=too-many-locals
+# pylint: disable=too-many-lines
+# pylint: disable=too-many-statements
+# pylint: disable=unused-argument
 
 import math
 import typing as T

diff --git a/...lass_in_values_test_data/python/symforce/codegen_test/codegen_dataclass_in_values_test.py b/...lass_in_values_test_data/python/symforce/codegen_test/codegen_dataclass_in_values_test.py
@@ -4,7 +4,11 @@
 # Do NOT modify by hand.
 # -----------------------------------------------------------------------------
 
-# pylint: disable=too-many-locals,too-many-lines,too-many-statements,unused-argument,unused-import
+# pylint: disable=useless-suppression
+# pylint: disable=too-many-locals
+# pylint: disable=too-many-lines
+# pylint: disable=too-many-statements
+# pylint: disable=unused-argument
 
 import math
 import typing as T

diff --git a/...symengine/codegen_python_test_data/python/symforce/codegen_python_test/python_function.py b/...symengine/codegen_python_test_data/python/symforce/codegen_python_test/python_function.py
@@ -4,7 +4,11 @@
 # Do NOT modify by hand.
 # -----------------------------------------------------------------------------
 
-# pylint: disable=too-many-locals,too-many-lines,too-many-statements,unused-argument,unused-import
+# pylint: disable=useless-suppression
+# pylint: disable=too-many-locals
+# pylint: disable=too-many-lines
+# pylint: disable=too-many-statements
+# pylint: disable=unused-argument
 
 import math
 import typing as T

diff --git a/...st_data/symengine/databuffer_codegen_test_data/python/symforce/buffer_test/buffer_func.py b/...st_data/symengine/databuffer_codegen_test_data/python/symforce/buffer_test/buffer_func.py
@@ -4,7 +4,11 @@
 # Do NOT modify by hand.
 # -----------------------------------------------------------------------------
 
-# pylint: disable=too-many-locals,too-many-lines,too-many-statements,unused-argument,unused-import
+# pylint: disable=useless-suppression
+# pylint: disable=too-many-locals
+# pylint: disable=too-many-lines
+# pylint: disable=too-many-statements
+# pylint: disable=unused-argument
 
 import math
 import typing as T

diff --git a/test/symforce_function_codegen_test_data/symengine/numba_test_func.py b/test/symforce_function_codegen_test_data/symengine/numba_test_func.py
@@ -4,7 +4,11 @@
 # Do NOT modify by hand.
 # -----------------------------------------------------------------------------
 
-# pylint: disable=too-many-locals,too-many-lines,too-many-statements,unused-argument,unused-import
+# pylint: disable=useless-suppression
+# pylint: disable=too-many-locals
+# pylint: disable=too-many-lines
+# pylint: disable=too-many-statements
+# pylint: disable=unused-argument
 
 import math
 import typing as T

diff --git a/...nction_codegen_test_data/symengine/symforce_pytorch_codegen_test/backend_test_function.py b/...nction_codegen_test_data/symengine/symforce_pytorch_codegen_test/backend_test_function.py
@@ -4,7 +4,11 @@
 # Do NOT modify by hand.
 # -----------------------------------------------------------------------------
 
-# pylint: disable=too-many-locals,too-many-lines,too-many-statements,unused-argument
+# pylint: disable=useless-suppression
+# pylint: disable=too-many-locals
+# pylint: disable=too-many-lines
+# pylint: disable=too-many-statements
+# pylint: disable=unused-argument
 
 import math  # pylint: disable=unused-import
 import typing as T

diff --git a/...mforce_function_codegen_test_data/symengine/symforce_pytorch_codegen_test/pytorch_func.py b/...mforce_function_codegen_test_data/symengine/symforce_pytorch_codegen_test/pytorch_func.py
@@ -4,7 +4,11 @@
 # Do NOT modify by hand.
 # -----------------------------------------------------------------------------
 
-# pylint: disable=too-many-locals,too-many-lines,too-many-statements,unused-argument
+# pylint: disable=useless-suppression
+# pylint: disable=too-many-locals
+# pylint: disable=too-many-lines
+# pylint: disable=too-many-statements
+# pylint: disable=unused-argument
 
 import math  # pylint: disable=unused-import
 import typing as T