Allow to apply unitary qubit operators to qubits #263
Conversation
| Example: | ||
|
|
||
| .. code-block:: python | ||
|
|
There was a problem hiding this comment.
This code is missing indentation I think
projectq/ops/_qubit_operator.py
Outdated
| raise TypeError("Only one qubit or qureg allowed.") | ||
| # Check that operator is unitary | ||
| if not len(self.terms) == 1: | ||
| raise TypeError("Only unitary QubitOperators can be applied to " |
There was a problem hiding this comment.
Only QubitOperators consisting of a single Pauli string with a coefficient of unit length can be applied.
Otherwise, people might get confused if their operator actually is unitary but consists of multiple terms.
There was a problem hiding this comment.
Can it be unitary with multiple terms?
There was a problem hiding this comment.
1/sqrt(2) * X + 1/sqrt(2) * Z = H
There was a problem hiding this comment.
Great, thanks for the example
projectq/ops/_qubit_operator.py
Outdated
| Return the inverse gate if QubitOperator is unitary. | ||
|
|
||
| Raises: | ||
| NotInvertible: inverse is not implemented if not unitary |
There was a problem hiding this comment.
if there are multiple terms or the coefficient is not 1 in absolute value
projectq/ops/_qubit_operator.py
Outdated
| Raises: | ||
| NotInvertible: inverse is not implemented if not unitary | ||
| """ | ||
| if len(self.terms) == 1: |
There was a problem hiding this comment.
and abs(coefficient)...
There was a problem hiding this comment.
It won't raise an exception now, though...
projectq/ops/_qubit_operator.py
Outdated
| Standard implementation of get_merged: | ||
|
|
||
| Raises: | ||
| NotMergeable: merging is not implemented |
There was a problem hiding this comment.
These docs are a bit strange ;-)
There was a problem hiding this comment.
yes, changed.
projectq/ops/_qubit_operator.py
Outdated
| len(other.terms) == 1 and | ||
| len(self.terms) == 1): | ||
| return self * other | ||
|
|
There was a problem hiding this comment.
Do we still need the comments below?
…ojectQ-Framework/ProjectQ into feature/unitary-qubitoperator-as-gate
No description provided.