unitaryfund / qrack

Comprehensive, GPU accelerated framework for developing universal virtual quantum processors
https://qrack.readthedocs.io/en/latest/
GNU Lesser General Public License v3.0
176 stars 38 forks source link

Cut redundancy over QStabilizer #955

Closed WrathfulSpatula closed 2 years ago

WrathfulSpatula commented 2 years ago

QStabilizer has recently been refactored to introspect and throw on cases of gates that it cannot apply. As a result, some QStabilizerHybrid implementation is redundant.

I notice that this PR itself adds a redundant check for separability on one control qubit in cases of non-Clifford MCMtrx(), and I notice that test_qft_cosmology benchmarks have recently slightly degraded, but any regression in performance in my tests doesn't seem to be a matter of this PR.