`measure_pauli_strings` not combining readout measurement circuits

Open eliottrosenberg opened this issue 6 months ago • 4 comments

https://github.com/quantumlib/Cirq/pull/7236 added the ability to specify groups of qubit-wise-commuting Pauli operators to measure simultaneously. Because these measurements are done simultaneously, readout on the qubits that these operators act on should also be measured simultaneously, but instead the code is generating a different readout circuit for each Pauli.

Jun 06 '25 00:06 eliottrosenberg

@ddddddanni

Jun 06 '25 00:06 eliottrosenberg

Discussed during Cirq Cynq 2025-06-11: @eliottrosenberg would you be able to provide a little bit more information?

Jun 11 '25 17:06 mhucka

I'm having https://github.com/quantumlib/Cirq/pull/7416 to fix this.

Jun 11 '25 17:06 ddddddanni

@mhucka happy to provide more context if you want, but it seems that @ddddddanni has it covered!

Jun 11 '25 17:06 eliottrosenberg

@eliottrosenberg @ddddddanni can/should this now be closed?

Sep 16 '25 17:09 daxfohl

@eliottrosenberg @ddddddanni can/should this now be closed?

Yes, this has been implemented. Could you help close it, since I don't have permission to do so? Thanks!

Sep 16 '25 17:09 ddddddanni