Enable pylint and ruff check for unused suppression comments (#7412)

No change in the effective code.

- ruff - fix RUF100 - unused-noqa directive
- pylint - fix useless-suppression
- pylint - remove unpaired pylint-enable directives
- pylint - enable `line-too-long` which was disabled above
- pylint - enable in `cirq.transformers.merge_k_qubit_gates_test`

Author: pavoljuhas

cirq-core/cirq/_compat_test.py

@@ -202,13 +202,11 @@ def old_func(*args, **kwargs):
         old_func(1, 2)
 
     with pytest.raises(AssertionError, match='deadline should match vX.Y'):
-        # pylint: disable=unused-variable
+
         @deprecated(deadline='invalid', fix='Roll some dice.')
         def badly_deprecated_func(*args, **kwargs):  # pragma: no cover
             return new_func(*args, **kwargs)
 
-        # pylint: enable=unused-variable
-
 
 def test_deprecated_parameter():
     @deprecated_parameter(
@@ -234,20 +232,16 @@ def f(new_count):
         'Double it yourself.',
         deadline='v1.2',
     ):
-        # pylint: disable=unexpected-keyword-arg
         # pylint: disable=no-value-for-parameter
         assert f(double_count=1) == 2
         # pylint: enable=no-value-for-parameter
-        # pylint: enable=unexpected-keyword-arg
 
     with pytest.raises(
         ValueError, match='During testing using Cirq deprecated functionality is not allowed'
     ):
-        # pylint: disable=unexpected-keyword-arg
         # pylint: disable=no-value-for-parameter
         f(double_count=1)
         # pylint: enable=no-value-for-parameter
-        # pylint: enable=unexpected-keyword-arg
 
     with pytest.raises(AssertionError, match='deadline should match vX.Y'):
 
@@ -259,12 +253,9 @@ def f(new_count):
             match=lambda args, kwargs: 'double_count' in kwargs,
             rewrite=lambda args, kwargs: (args, {'new_count': kwargs['double_count'] * 2}),
         )
-        # pylint: disable=unused-variable
         def f_with_badly_deprecated_param(new_count):  # pragma: no cover
             return new_count
 
-        # pylint: enable=unused-variable
-
 
 @duet.sync
 async def test_deprecated_parameter_async_function():
@@ -293,11 +284,9 @@ async def f(new_count):
         'Double it yourself.',
         deadline='v1.2',
     ):
-        # pylint: disable=unexpected-keyword-arg
         # pylint: disable=no-value-for-parameter
         assert await f(double_count=1) == 2
         # pylint: enable=no-value-for-parameter
-        # pylint: enable=unexpected-keyword-arg
 
 
 def test_wrap_module():
@@ -587,7 +576,6 @@ def _repeated_import_path():
 
 def _type_repr_in_deprecated_module():
     # initialize the DeprecatedModuleFinders
-    # pylint: disable=unused-import
     import cirq.testing._compat_test_data.fake_a as mod_a
 
     expected_repr = "<class 'cirq.testing._compat_test_data.module_a.types.SampleType'>"
@@ -763,8 +751,6 @@ def test_metadata_search_path():
 
 def _test_metadata_search_path_inner():  # pragma: no cover
     # initialize the DeprecatedModuleFinders
-    # pylint: disable=unused-import
-
     assert importlib.metadata.metadata('numpy')
 
 
@@ -805,7 +791,6 @@ def test_type_repr_in_new_module():
 
 def _test_type_repr_in_new_module_inner():
     # initialize the DeprecatedModuleFinders
-    # pylint: disable=unused-import
     import cirq.testing._compat_test_data.module_a as mod_a
 
     expected_repr = "<class 'cirq.testing._compat_test_data.module_a.types.SampleType'>"
@@ -875,7 +860,6 @@ def test_new_module_is_top_level():
 
 def _test_new_module_is_top_level_inner():
     # sets up the DeprecationFinders
-    # pylint: disable=unused-import
     import time
 
     # imports a top level module that was also deprecated

cirq-core/cirq/circuits/circuit.py

@@ -233,7 +233,6 @@ def _decompose_(self) -> cirq.OP_TREE:
         """See `cirq.SupportsDecompose`."""
         return self.all_operations()
 
-    # pylint: disable=function-redefined
     @overload
     def __getitem__(self, key: int) -> cirq.Moment:
         pass
@@ -277,8 +276,6 @@ def __getitem__(self, key):
 
         raise TypeError('__getitem__ called with key not of type slice, int, or tuple.')
 
-    # pylint: enable=function-redefined
-
     def __str__(self) -> str:
         return self.to_text_diagram()
 
@@ -1895,7 +1892,6 @@ def copy(self) -> Circuit:
         copied_circuit._placement_cache = None
         return copied_circuit
 
-    # pylint: disable=function-redefined
     @overload
     def __setitem__(self, key: int, value: cirq.Moment):
         pass
@@ -1916,8 +1912,6 @@ def __setitem__(self, key, value):
         self._moments[key] = value
         self._mutated()
 
-    # pylint: enable=function-redefined
-
     def __delitem__(self, key: int | slice):
         del self._moments[key]
         self._mutated()

cirq-core/cirq/circuits/moment.py

@@ -538,7 +538,6 @@ def __sub__(self, other: cirq.OP_TREE) -> cirq.Moment:
             )
         return Moment(new_ops)
 
-    # pylint: disable=function-redefined
     @overload
     def __getitem__(self, key: raw_types.Qid) -> cirq.Operation:
         pass

cirq-core/cirq/contrib/acquaintance/bipartite_test.py

@@ -164,9 +164,11 @@
                 │   │         │   │
 3: ─────────────█───1↦0───────█───1↦0─────────────
 """,
-    ('decomposed', cca.BipartiteGraphType.COMPLETE, 3):
-    # pylint: disable=line-too-long
-    """
+    (
+        'decomposed',
+        cca.BipartiteGraphType.COMPLETE,
+        3,
+    ): """
 0: ───────────────────────█───0↦1───────────────────────────█───0↦1───────────────────────
                           │   │                             │   │
 1: ─────────────█───0↦1───█───1↦0───█───0↦1───────█───0↦1───█───1↦0───█───0↦1─────────────
@@ -202,7 +204,6 @@
 7: ─────────────────────────────────█───1↦0───────────────────────────────────────────────█───1↦0─────────────────────────────────
 
 """,
-    # pylint: enable=line-too-long
     (
         'decomposed',
         cca.BipartiteGraphType.MATCHING,

cirq-core/cirq/contrib/quirk/quirk_gate.py

@@ -113,7 +113,6 @@ def single_qubit_matrix_gate(matrix: np.ndarray | None) -> QuirkOp | None:
     if matrix is None or matrix.shape[0] != 2:
         return None
 
-    # pylint: disable=consider-using-f-string
     matrix = matrix.round(6)
     matrix_repr = '{{%s+%si,%s+%si},{%s+%si,%s+%si}}' % (
         np.real(matrix[0, 0]),

cirq-core/cirq/linalg/decompositions.py

@@ -130,7 +130,6 @@ def unitary_eig(
     return R.diagonal(), V
 
 
-# pylint: enable=missing-raises-doc
 def map_eigenvalues(
     matrix: np.ndarray, func: Callable[[complex], complex], *, atol: float = 1e-8
 ) -> np.ndarray:

cirq-core/cirq/ops/clifford_gate.py

@@ -747,11 +747,7 @@ def __pow__(self, exponent: float) -> SingleQubitCliffordGate:
 
         return NotImplemented
 
-    def _act_on_(
-        self,
-        sim_state: cirq.SimulationStateBase,  # pylint: disable=unused-argument
-        qubits: Sequence[cirq.Qid],  # pylint: disable=unused-argument
-    ):
+    def _act_on_(self, sim_state: cirq.SimulationStateBase, qubits: Sequence[cirq.Qid]):
         # TODO(#5256) Add the implementation of _act_on_ with CliffordTableauSimulationState.
         return NotImplemented
 

cirq-core/cirq/ops/eigen_gate.py

@@ -128,11 +128,9 @@ def _with_exponent(self, exponent: value.TParamVal) -> EigenGate:
         Child classes should override this method if they have an __init__
         method with a differing signature.
         """
-        # pylint: disable=unexpected-keyword-arg
         if self._global_shift == 0:
             return type(self)(exponent=exponent)
         return type(self)(exponent=exponent, global_shift=self._global_shift)
-        # pylint: enable=unexpected-keyword-arg
 
     def _diagram_exponent(
         self, args: protocols.CircuitDiagramInfoArgs, *, ignore_global_phase: bool = True

cirq-core/cirq/ops/measure_util.py

@@ -90,9 +90,6 @@ def measure_paulistring_terms(
     return [PauliMeasurementGate([pauli_basis[q]], key=key_func(q)).on(q) for q in pauli_basis]
 
 
-# pylint: disable=function-redefined
-
-
 @overload
 def measure(
     *target: raw_types.Qid,
@@ -205,6 +202,3 @@ def measure_each(
     )
     qubitsequence = qubits[0] if one_iterable_arg else qubits
     return [MeasurementGate(1, key_func(q), qid_shape=(q.dimension,)).on(q) for q in qubitsequence]
-
-
-# pylint: enable=function-redefined

cirq-core/cirq/ops/pauli_string.py

@@ -232,7 +232,6 @@ def equal_up_to_coefficient(self, other: cirq.PauliString) -> bool:
     def __getitem__(self, key: TKey) -> pauli_gates.Pauli:
         return self._qubit_pauli_map[key]
 
-    # pylint: disable=function-redefined
     @overload
     def get(self, key: Any, default: None = None) -> pauli_gates.Pauli | None:
         pass
@@ -281,8 +280,6 @@ def __mul__(self, other):
             )
         return NotImplemented
 
-    # pylint: enable=function-redefined
-
     @property
     def gate(self) -> cirq.DensePauliString:
         """Returns a `cirq.DensePauliString`"""
@@ -897,9 +894,7 @@ def dense(self, qubits: Sequence[TKey]) -> cirq.DensePauliString:
 
         if not self.keys() <= set(qubits):
             raise ValueError('not self.keys() <= set(qubits)')
-        # pylint: disable=too-many-function-args
         pauli_mask = [self.get(q, identity.I) for q in qubits]
-        # pylint: enable=too-many-function-args
         return dense_pauli_string.DensePauliString(pauli_mask, coefficient=self.coefficient)
 
     def conjugated_by(self, clifford: cirq.OP_TREE) -> PauliString:
@@ -1349,7 +1344,6 @@ def __setitem__(self, key: TKey, value: cirq.PAULI_GATE_LIKE):
     def __delitem__(self, key: TKey):
         del self.pauli_int_dict[key]
 
-    # pylint: disable=function-redefined
     @overload
     def get(self, key: TKey, default: None = None) -> cirq.Pauli | None:
         pass
@@ -1363,7 +1357,6 @@ def get(self, key: TKey, default=None) -> cirq.Pauli | TDefault | None:
         result = self.pauli_int_dict.get(key, None)
         return default if result is None else _INT_TO_PAULI[result - 1]
 
-    # pylint: enable=function-redefined
     def inplace_before(self, ops: cirq.OP_TREE) -> cirq.MutablePauliString:
         r"""Propagates the pauli string from after to before a Clifford effect.