For discussion: use black for code formatting

.pylintrc

@@ -70,7 +70,8 @@ disable=no-self-use,        # disabled as it is too verbose
     unnecessary-pass,   # allow for methods with just "pass", for clarity
     no-else-return,     # relax "elif" after a clause with a return
     docstring-first-line-empty, # relax docstring style
-    import-outside-toplevel
+    import-outside-toplevel,
+    bad-continuation, bad-whitespace # differences of opinion with black
 
 
 
@@ -211,7 +212,7 @@ max-nested-blocks=5
 [FORMAT]
 
 # Maximum number of characters on a single line.
-max-line-length=100
+max-line-length=105
 
 # Regexp for a line that is allowed to be longer than the limit.
 ignore-long-lines=^\s*(# )?<?https?://\S+>?$

pyproject.toml

@@ -1,2 +1,6 @@
 [build-system]
 requires = ["Cython>=0.27.1", "setuptools", "wheel"]
+
+[tool.black]
+line-length = 100
+target-version = ['py36', 'py37', 'py38']

qiskit/__init__.py

@@ -64,7 +64,9 @@
     warnings.warn(
         "Using Qiskit with Python 3.6 is deprecated as of the 0.17.0 release. "
         "Support for running Qiskit with Python 3.6 will be removed in a "
-        "future release.", DeprecationWarning)
+        "future release.",
+        DeprecationWarning,
+    )
 
 
 class AerWrapper:
@@ -77,6 +79,7 @@ def __bool__(self):
         if self.aer is None:
             try:
                 from qiskit.providers import aer
+
                 self.aer = aer.Aer
             except ImportError:
                 return False
@@ -86,11 +89,14 @@ def __getattr__(self, attr):
         if not self.aer:
             try:
                 from qiskit.providers import aer
+
                 self.aer = aer.Aer
             except ImportError as exc:
-                raise ImportError('Could not import the Aer provider from the '
-                                  'qiskit-aer package. Install qiskit-aer or '
-                                  'check your installation.') from exc
+                raise ImportError(
+                    "Could not import the Aer provider from the "
+                    "qiskit-aer package. Install qiskit-aer or "
+                    "check your installation."
+                ) from exc
         return getattr(self.aer, attr)
 
 
@@ -104,6 +110,7 @@ def __bool__(self):
         if self.ibmq is None:
             try:
                 from qiskit.providers import ibmq
+
                 self.ibmq = ibmq.IBMQ
             except ImportError:
                 return False
@@ -113,12 +120,15 @@ def __getattr__(self, attr):
         if not self.ibmq:
             try:
                 from qiskit.providers import ibmq
+
                 self.ibmq = ibmq.IBMQ
             except ImportError as exc:
-                raise ImportError('Could not import the IBMQ provider from the '
-                                  'qiskit-ibmq-provider package. Install '
-                                  'qiskit-ibmq-provider or check your  '
-                                  'installation.') from exc
+                raise ImportError(
+                    "Could not import the IBMQ provider from the "
+                    "qiskit-ibmq-provider package. Install "
+                    "qiskit-ibmq-provider or check your  "
+                    "installation."
+                ) from exc
         return getattr(self.ibmq, attr)
 
 

qiskit/algorithms/__init__.py

@@ -130,48 +130,58 @@
 from .variational_algorithm import VariationalAlgorithm, VariationalResult
 from .amplitude_amplifiers import Grover, GroverResult, AmplificationProblem
 from .amplitude_estimators import (
-    AmplitudeEstimator, AmplitudeEstimatorResult,
-    AmplitudeEstimation, AmplitudeEstimationResult,
-    FasterAmplitudeEstimation, FasterAmplitudeEstimationResult,
-    IterativeAmplitudeEstimation, IterativeAmplitudeEstimationResult,
-    MaximumLikelihoodAmplitudeEstimation, MaximumLikelihoodAmplitudeEstimationResult,
-    EstimationProblem
+    AmplitudeEstimator,
+    AmplitudeEstimatorResult,
+    AmplitudeEstimation,
+    AmplitudeEstimationResult,
+    FasterAmplitudeEstimation,
+    FasterAmplitudeEstimationResult,
+    IterativeAmplitudeEstimation,
+    IterativeAmplitudeEstimationResult,
+    MaximumLikelihoodAmplitudeEstimation,
+    MaximumLikelihoodAmplitudeEstimationResult,
+    EstimationProblem,
 )
 from .eigen_solvers import NumPyEigensolver, Eigensolver, EigensolverResult
 from .factorizers import Shor, ShorResult
-from .minimum_eigen_solvers import (VQE, VQEResult, QAOA,
-                                    NumPyMinimumEigensolver,
-                                    MinimumEigensolver, MinimumEigensolverResult)
+from .minimum_eigen_solvers import (
+    VQE,
+    VQEResult,
+    QAOA,
+    NumPyMinimumEigensolver,
+    MinimumEigensolver,
+    MinimumEigensolverResult,
+)
 from .exceptions import AlgorithmError
 
 __all__ = [
-    'AlgorithmResult',
-    'VariationalAlgorithm',
-    'VariationalResult',
-    'AmplificationProblem',
-    'Grover',
-    'GroverResult',
-    'AmplitudeEstimator',
-    'AmplitudeEstimatorResult',
-    'AmplitudeEstimation',
-    'AmplitudeEstimationResult',
-    'FasterAmplitudeEstimation',
-    'FasterAmplitudeEstimationResult',
-    'IterativeAmplitudeEstimation',
-    'IterativeAmplitudeEstimationResult',
-    'MaximumLikelihoodAmplitudeEstimation',
-    'MaximumLikelihoodAmplitudeEstimationResult',
-    'EstimationProblem',
-    'NumPyEigensolver',
-    'Eigensolver',
-    'EigensolverResult',
-    'Shor',
-    'ShorResult',
-    'VQE',
-    'VQEResult',
-    'QAOA',
-    'NumPyMinimumEigensolver',
-    'MinimumEigensolver',
-    'MinimumEigensolverResult',
-    'AlgorithmError',
+    "AlgorithmResult",
+    "VariationalAlgorithm",
+    "VariationalResult",
+    "AmplificationProblem",
+    "Grover",
+    "GroverResult",
+    "AmplitudeEstimator",
+    "AmplitudeEstimatorResult",
+    "AmplitudeEstimation",
+    "AmplitudeEstimationResult",
+    "FasterAmplitudeEstimation",
+    "FasterAmplitudeEstimationResult",
+    "IterativeAmplitudeEstimation",
+    "IterativeAmplitudeEstimationResult",
+    "MaximumLikelihoodAmplitudeEstimation",
+    "MaximumLikelihoodAmplitudeEstimationResult",
+    "EstimationProblem",
+    "NumPyEigensolver",
+    "Eigensolver",
+    "EigensolverResult",
+    "Shor",
+    "ShorResult",
+    "VQE",
+    "VQEResult",
+    "QAOA",
+    "NumPyMinimumEigensolver",
+    "MinimumEigensolver",
+    "MinimumEigensolverResult",
+    "AlgorithmError",
 ]

qiskit/algorithms/algorithm_result.py

@@ -26,15 +26,18 @@ class AlgorithmResult(ABC):
     def __str__(self) -> str:
         result = OrderedDict()
         for name, value in inspect.getmembers(self):
-            if not name.startswith('_') and \
-                    not inspect.ismethod(value) and not inspect.isfunction(value) and \
-                    hasattr(self, name):
+            if (
+                not name.startswith("_")
+                and not inspect.ismethod(value)
+                and not inspect.isfunction(value)
+                and hasattr(self, name)
+            ):
 
                 result[name] = value
 
         return pprint.pformat(result, indent=4)
 
-    def combine(self, result: 'AlgorithmResult') -> None:
+    def combine(self, result: "AlgorithmResult") -> None:
         """
         Any property from the argument that exists in the receiver is
         updated.
@@ -44,13 +47,16 @@ def combine(self, result: 'AlgorithmResult') -> None:
             TypeError: Argument is None
         """
         if result is None:
-            raise TypeError('Argument result expected.')
+            raise TypeError("Argument result expected.")
         if result == self:
             return
 
         # find any result public property that exists in the receiver
         for name, value in inspect.getmembers(result):
-            if not name.startswith('_') and \
-                    not inspect.ismethod(value) and not inspect.isfunction(value) and \
-                    hasattr(self, name):
+            if (
+                not name.startswith("_")
+                and not inspect.ismethod(value)
+                and not inspect.isfunction(value)
+                and hasattr(self, name)
+            ):
                 setattr(self, name, value)

qiskit/algorithms/amplitude_amplifiers/__init__.py

@@ -17,9 +17,9 @@
 from .grover import Grover, GroverResult
 
 __all__ = [
-    'AmplitudeAmplifier',
-    'AmplitudeAmplifierResult',
-    'AmplificationProblem',
-    'Grover',
-    'GroverResult'
+    "AmplitudeAmplifier",
+    "AmplitudeAmplifierResult",
+    "AmplificationProblem",
+    "Grover",
+    "GroverResult",
 ]

qiskit/algorithms/amplitude_amplifiers/amplification_problem.py

@@ -27,14 +27,15 @@ class AmplificationProblem:
     on the optimal bitstring.
     """
 
-    def __init__(self,
-                 oracle: QuantumCircuit,
-                 state_preparation: Optional[QuantumCircuit] = None,
-                 grover_operator: Optional[QuantumCircuit] = None,
-                 post_processing: Optional[Callable[[str], Any]] = None,
-                 objective_qubits: Optional[Union[int, List[int]]] = None,
-                 is_good_state: Optional[Callable[[str], bool]] = None,
-                 ) -> None:
+    def __init__(
+        self,
+        oracle: QuantumCircuit,
+        state_preparation: Optional[QuantumCircuit] = None,
+        grover_operator: Optional[QuantumCircuit] = None,
+        post_processing: Optional[Callable[[str], Any]] = None,
+        objective_qubits: Optional[Union[int, List[int]]] = None,
+        is_good_state: Optional[Callable[[str], bool]] = None,
+    ) -> None:
         r"""
         Args:
             oracle: The oracle reflecting about the bad states.
@@ -158,15 +159,17 @@ def is_good_state(self) -> Callable[[str], float]:
             if all(isinstance(good_bitstr, str) for good_bitstr in self._is_good_state):
                 return lambda bitstr: bitstr in self._is_good_state
             else:
-                return lambda bitstr: all(bitstr[good_index] == '1'  # type:ignore
-                                          for good_index in self._is_good_state)
+                return lambda bitstr: all(
+                    bitstr[good_index] == "1"  # type:ignore
+                    for good_index in self._is_good_state
+                )
 
         return lambda bitstr: bitstr in self._is_good_state.probabilities_dict()
 
     @is_good_state.setter
-    def is_good_state(self,
-                      is_good_state: Union[Callable[[str], bool], List[int], List[str], Statevector]
-                      ) -> None:
+    def is_good_state(
+        self, is_good_state: Union[Callable[[str], bool], List[int], List[str], Statevector]
+    ) -> None:
         """Set the ``is_good_state`` function.
 
         Args:

qiskit/algorithms/amplitude_amplifiers/amplitude_amplifier.py

@@ -23,7 +23,7 @@ class AmplitudeAmplifier(ABC):
     """The interface for amplification algorithms."""
 
     @abstractmethod
-    def amplify(self, amplification_problem: AmplificationProblem) -> 'AmplificationResult':
+    def amplify(self, amplification_problem: AmplificationProblem) -> "AmplificationResult":
         """Run the amplification algorithm.
 
         Args: