Qiskit / qiskit / 23738581156

87%

Build Type

push

github

Committed by web-flow

Commit Message

Support commutation check between Pauli-based gates and standard gates (#15488)

* Transpiler: implement exact Pauli generators for LightCone in Rust

This PR adds exact Pauli generator expansions for the following gates:
- single-qubit: H (X+Z)
- 2-qubit: CX, CY, CZ, ECR, Swap, iSwap, CSX, CS, CSdg
- parametric: CRX, CRY, CRZ, CPhase, RXX, RYY, RZZ, RZX, XXPlusYY, XXMinusYY
- 3-qubit: CCX, CSwap

This enables high-performance bitwise commutation checks in the LightCone
pass, avoiding slow matrix-multiplication fallbacks. Parametric gates are
scaled dynamically in Rust using the input parameters.

Co-authored-by: Shelly Garion <shelly.garion@ibm.com>
Co-authored-by: Alexander Ivrii <alexander.ivrii@ibm.com>

* Lint: apply cargo fmt and fix clippy::map_clone in split_2q_unitaries.rs

* Expose generator_observable to Python

Following feedback from Alexander Ivrii, this commit exposes the Rust
generator_observable function and the StandardGate enum to Python under
the qiskit._accelerate.standard_generators submodule.

This allows developers to verify the mathematical generators directly from
Python and reuse the logic for other purposes like PauliEvolutionGate
verification. The Rust-internal API is maintained via a dedicated
generator_observable function that accepts slices.

* Refactor generator logic to use qiskit_circuit::StandardGate

* Update generator_observable with expansions for CCX/CCZ/CSwap

* Fix generator fallback for unsupported gates and update docs

* Narrow PR scope: remove split_2q_unitaries and pyext python wrapper

* Add generator support for CS, CSdg, CSX, CCX, CCZ; add commutation tests

* style: run black on test_commutation_checker.py

* style: run rustfmt on standard_generators.rs

* lint: fix license headers and pylint warnings

* style: remove decorative unicode lines and add operator equivalence tests

* fix: ensure strict Operator equivalence for all supported generators

* style: fix pylint reimport and missing variables in... (continued)

Run Details

282 of 295 new or added lines in 5 files covered. (95.59%)

21 existing lines in 4 files now uncovered.

104300 of 119210 relevant lines covered (87.49%)

1018997.11 hits per line