Qiskit / qiskit / 32853599859

88%

main: 88%

Build Type

push

github

Committed by web-flow

Commit Message

Fix spurious clone on `SparsePauliOp.to_matrix(sparse=False)` (#16828)

* Fix spurious clone on `SparsePauliOp.to_matrix(sparse=False)`

We always own the array; there was never a need to clone out to Python
space.  The actual performance improvement varies a lot depending on the
workload, the parallelism in the algorithm, and performance of the
memory allocator.  The clone was always done single-threaded previously,
so speedups in excess of 2x are quite reasonable.

* Reframe as bugfix

This isn't a longstanding issue; it was introduced accidentally in the
v2.5 cycle[^1].

[^1]: 230a09281b: Add `PauliProductRotation.matrix` (#15975)

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

22 existing lines in 4 files now uncovered.

114173 of 130119 relevant lines covered (87.75%)

989924.96 hits per line