Remove useless pylint suppressions (#8292)

**Context:**

**Description of the Change:**

**Benefits:**

**Possible Drawbacks:**

**Related GitHub Issues:**

Author: astralcai

pennylane/allocation.py

@@ -47,7 +47,6 @@ class AllocateState(StrEnum):
     allocate_prim = QmlPrimitive("allocate")
     allocate_prim.multiple_results = True
 
-    # pylint: disable=unused-argument
     @allocate_prim.def_impl
     def _(*, num_wires, state: AllocateState = AllocateState.ZERO, restored=False):
         raise NotImplementedError("jaxpr containing qubit allocation cannot be executed.")
@@ -60,7 +59,6 @@ def _(*, num_wires, state: AllocateState = AllocateState.ZERO, restored=False):
     deallocate_prim = QmlPrimitive("deallocate")
     deallocate_prim.multiple_results = True
 
-    # pylint: disable=unused-argument
     @deallocate_prim.def_impl
     def _(*wires):
         raise NotImplementedError("jaxpr containing qubit deallocation cannot be executed.")
@@ -171,7 +169,6 @@ def c():
     return Deallocate(wires)
 
 
-# pylint: disable=too-many-ancestors
 class DynamicRegister(Wires):
     """A specialized ``Wires`` class for dynamic wires with a context manager for automatic deallocation."""
 

pennylane/bose/bosonic.py

@@ -413,7 +413,6 @@ def shift_operator(self, initial_position, final_position):
         return bs
 
 
-# pylint: disable=useless-super-delegation
 class BoseSentence(dict):
     r"""Dictionary used to represent a Bose sentence, a linear combination of Bose words,
     with the keys as BoseWord instances and the values correspond to coefficients.

pennylane/capture/autograph/operator_update.py

@@ -13,13 +13,14 @@
 # limitations under the License.
 """Converter for array element operator assignment."""
 
-# pylint: disable=no-member, too-few-public-methods
+# pylint: disable=no-member
 
 import gast
 from malt.core import converter
 from malt.pyct import templates
 
 
+# pylint: disable=too-few-public-methods
 # TODO: The methods from this class should be moved to the SliceTransformer class in DiastaticMalt
 class SingleIndexArrayOperatorUpdateTransformer(converter.Base):
     """Converts array element operator assignment statements into calls to update_item_with_{op},

pennylane/capture/base_interpreter.py

@@ -14,11 +14,11 @@
 """
 This submodule defines a strategy structure for defining custom plxpr interpreters
 """
+# pylint: disable=no-self-use
+
 from collections.abc import Callable, Sequence
 from copy import copy
 from functools import partial, wraps
-
-# pylint: disable=no-self-use, wrong-import-position
 from importlib.metadata import version
 
 import jax
@@ -645,7 +645,6 @@ class FlattenedInterpreter(PlxprInterpreter):
     from jax._src.pjit import pjit_p
 
 
-# pylint: disable=protected-access
 @FlattenedInterpreter.register_primitive(pjit_p)
 def _(self, *invals, jaxpr, **params):
     if jax.config.jax_dynamic_shapes:

pennylane/capture/custom_primitives.py

@@ -30,7 +30,7 @@ class PrimitiveType(Enum):
     TRANSFORM = "transform"
 
 
-# pylint: disable=too-few-public-methods
+# pylint: disable=abstract-method,too-few-public-methods
 class QmlPrimitive(Primitive):
     """A subclass for JAX's Primitive that differentiates between different
     classes of primitives."""

pennylane/compiler/python_compiler/dialects/quantum.py

@@ -511,7 +511,6 @@ class GlobalPhaseOp(IRDLOperation):
 
     out_ctrl_qubits = var_result_def(QubitType)
 
-    # pylint: disable=too-many-arguments
     def __init__(
         self,
         *,

pennylane/compiler/python_compiler/dialects/stablehlo/data_movement.py

@@ -92,7 +92,7 @@ def verify_(self) -> None:
         assert isinstance(o_type, TensorType) and isinstance(r_type, TensorType)
 
         # broadcast_in_dim_c2: broadcast_dimensions size == operand rank
-        dims = tuple(self.broadcast_dimensions.get_values())  # pylint: disable=no-member
+        dims = tuple(self.broadcast_dimensions.get_values())
         operand_rank = o_type.get_num_dims()
         if len(dims) != operand_rank:
             raise VerifyException(

pennylane/compiler/python_compiler/dialects/stablehlo/types.py

@@ -48,8 +48,6 @@
     NestedTupleOfConstraint,
 )
 
-# pylint: disable=fixme
-
 
 def _create_param_constrained_type(
     base_attr: type, widths: list[int], signedness: Signedness | None = None

pennylane/compiler/python_compiler/dialects/transform.py

@@ -41,7 +41,7 @@
 
 from xdsl.dialects.builtin import Dialect
 
-# pylint: disable=unused-wildcard-import,wildcard-import,undefined-variable,too-few-public-methods
+# pylint: disable=too-few-public-methods
 from xdsl.dialects.transform import ApplyRegisteredPassOp as xApplyRegisteredPassOp
 from xdsl.dialects.transform import (
     DictionaryAttr,
@@ -60,7 +60,6 @@
 
 
 @irdl_op_definition
-# pylint: disable=function-redefined
 class ApplyRegisteredPassOp(IRDLOperation):
     """
     See external [documentation](https://mlir.llvm.org/docs/Dialects/Transform/#transformapply_registered_pass-transformapplyregisteredpassop).

pennylane/compiler/python_compiler/jax_utils.py

@@ -20,7 +20,7 @@
 
 from catalyst import QJIT
 from jax._src.lib import _jax
-from jaxlib.mlir.dialects import stablehlo as jstablehlo  # pylint: disable=no-name-in-module
+from jaxlib.mlir.dialects import stablehlo as jstablehlo
 from jaxlib.mlir.ir import Context as jContext  # pylint: disable=no-name-in-module
 from jaxlib.mlir.ir import Module as jModule  # pylint: disable=no-name-in-module
 from xdsl.context import Context as xContext
@@ -38,7 +38,7 @@
 JaxJittedFunction: TypeAlias = _jax.PjitFunction  # pylint: disable=c-extension-no-member
 
 
-class QuantumParser(xParser):  # pylint: disable=abstract-method,too-few-public-methods
+class QuantumParser(xParser):  # pylint: disable=too-few-public-methods
     """A subclass of ``xdsl.parser.Parser`` that automatically loads relevant dialects
     into the input context.