phase exponent ignored in approximate comparisons of cirq.PhasedISwapPowGate
#6529
Labels
good first issue
This issue can be resolved by someone who is not familiar with the codebase. A good starting issue.
kind/bug-report
Something doesn't seem to work.
no QC knowledge needed
Want to contribute to Cirq, but don't know quantum computing? This issue is for you.
triage/accepted
there is consensus amongst maintainers that this is a real bug or a reasonable feature to add
Description of the issue
PhasedISwapPowGate
doesn't override the_value_equality_approximate_values_
implementation it inherits fromEigenGate
, which means its phase exponent attribute isn't included in approximate comparisonsthe one exception is when its phase exponent is zero, in which case approximate comparison fails even when it shouldn't due to some of the same inconsistencies described for
cirq.PhasedXPowGate
in #6528How to reproduce the issue
Cirq version
The text was updated successfully, but these errors were encountered: