chore: review

Author: Henri-ColibrITD

mpqp/core/circuit.py

@@ -173,9 +173,7 @@ def __init__(
             self.add(deepcopy(data))
 
     def __eq__(self, value: object) -> bool:
-        if not isinstance(value, QCircuit):
-            return False
-        return self.to_dict() == value.to_dict()
+        return isinstance(value, type(self)) and self.to_dict() == value.to_dict()
 
     def add(self, components: OneOrMany[Instruction | NoiseModel]):
         """Adds a ``component`` or a list of ``component`` at the end of the

mpqp/core/instruction/barrier.py

@@ -43,6 +43,4 @@ def to_other_language(
             raise NotImplementedError(f"{language} is not supported")
 
     def __repr__(self):
-        if self._dynamic:
-            return f"{type(self).__name__}()"
-        return f"{type(self).__name__}({self.size})"
+        return f"{type(self).__name__}({'' if self._dynamic else self.size})"

mpqp/core/instruction/instruction.py

@@ -103,11 +103,7 @@ def __eq__(self, value: object) -> bool:
     def __str__(self) -> str:
         from mpqp.core.circuit import QCircuit
 
-        connection = self.connections()
-        circuit_size = max(connection) + 1 if len(connection) != 0 else 1
-        circuit = QCircuit(circuit_size)
-        circuit.add(self)
-        return str(circuit)
+        return str(QCircuit([self], nb_qubits=max(self.connections(), default=0) + 1))
 
     def __repr__(self) -> str:
         from mpqp.core.instruction.gates import ControlledGate

mpqp/local_storage/load.py

@@ -1,7 +1,6 @@
-"""provides utility functions for converting job and result data from the local
-storage format into MPQP (:class:`~mpqp.execution.job.Job` and :class:`~mpqp.execution.result.Result`) objects.
-It also includes functions to query and retrieve jobs and results from the database.
-"""
+"""This module provides utility functions retrieving jobs and results from local 
+storage. In the process, they are converted to MPQP  objects 
+(:class:`~mpqp.execution.job.Job` and :class:`~mpqp.execution.result.Result`)."""
 
 from __future__ import annotations
 
@@ -160,11 +159,11 @@ def get_all_results() -> list[Result]:
 
 
 def get_jobs_with_job(job: Job | list[Job]) -> list[Job]:
-    """Retrieve job(s) matching the given job(s) attributes from the database:
-        - JobType
-        - Circuit
-        - Device
-        - Measure
+    """Retrieve job(s) matching the given job(s) attributes from the database
+        - ``JobType``
+        - ``Circuit``
+        - ``Device``
+        - ``Measure``
 
     Args:
         job: Job(s) to search for.

mpqp/local_storage/queries.py

@@ -12,7 +12,7 @@
 from mpqp.execution.connection.env_manager import get_env_variable
 from mpqp.execution.job import Job
 from mpqp.execution.result import BatchResult, Result
-from mpqp.local_storage.setup import ensure_local_storage, DictDB
+from mpqp.local_storage.setup import DictDB, ensure_local_storage
 
 
 @dataclass

mpqp/tools/maths.py

@@ -102,10 +102,7 @@ def is_hermitian(matrix: Matrix) -> bool:
         False
 
     """
-    return matrix_eq(
-        np.array(matrix).transpose().conjugate(),  # pyright: ignore[reportArgumentType]
-        matrix,
-    )
+    return matrix_eq(np.array(matrix).transpose().conjugate(), matrix)
 
 
 @typechecked