Qiskit / qiskit / 11723831465

88%

Build Type

push

github

Committed by web-flow

Commit Message

Add pauli_twirl_2q_gates function (#13331)

* Add twirl_circuit function

This commit adds a new function twirl_circuit to apply Pauli twirling to
a given circuit. This function only works with a fixed set of two qubit
gates and is not a general solution. For performance this new function
is written in rust and uses static lookup tables for the twirling sets
around the fixed two qubit gates that are randomly sampled. There is
an option to perform twirling multiple times and return a list of
circuits instead of just doing it once. Ideally we'd do this in
parallel, but we're currently blocked on the use of OnceCell in the
PackedInstruction and the ParameterTable from parallelizing with
CircuitData. We can further improve the performance of this new
function by using a parallel iterator once #13219 is resolved. The
function is written in a way to make this simple in the future.

Fixes #13325

Co-authored-by: Paul Nation <nonhermitian@gmail.com>

* Update asv benchmarks to use new function

* Apply suggestions from code review

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Expand testing coverage

* Add capacity option to clone_empty_from

* Make static twirling tables more compact

* Make num_trials default to None to return a single circuit

* Allow for multiple twirling gates

This commit expands the twirling_gate argument to work with strings
instead of classes and also specify a list of gates instead of a single
gate type. It also changes the default to `None` to twirl all supported
gates in the circuit.

* Fix typo in max seed value

* Avoid vec for qubits

* Recurse into control flow for twirling

This commit updates the twirl_circuits logic to recurse into a control
flow operation and twirl any gates that are potentially in the block.

* Avoid extra calls to interner

Previously for each twirled gate we were calling the interner 4 times
once for each new gate. However, there are only 2 qargs being used and
we only need half of the interning ... (continued)

Run Details

398 of 410 new or added lines in 8 files covered. (97.07%)

15 existing lines in 2 files now uncovered.

78195 of 88004 relevant lines covered (88.85%)

365150.01 hits per line