1
88%
main: 88%

Ran 07 Nov 2025 02:05PM UTC

Files 875

Run time 38s

Badge

Embed ▾

Committed 07 Nov 2025 01:00PM UTC coverage: 88.162% (-0.002%) from 88.164%

Job # 19169744603.1

Build Type

push

github

Committed by

web-flow

Commit Message

Skip intermediate PyList in PauliProductMeasurement::create_py_op() (#15310)

In the recently merged #15126 the PauliProductMeasurement operation
being added to Rust was unecessarily creating a pylist to go from
`Vec<bool>` -> `np.array[bool]` for the z and x parameters while we can
leverage rust-numpy to go straight from a Vec to a numpy array. This
was raised in review for the PR, but the fix just changed the mechanism
a Pylist was created by and didn't actually remove the python list
creation. `into_bound_py_any()` just is a short cut for using the
default IntoPyObject trait's conversion and casting it as a PyAny. In
the case of `Vec<T>` that means it's converting to a `PyList<T>`. Doing
this still results in 2 copies, one of which is done in Python. This
commit fixes the issue by using `to_pyarray()` to copy the Vec<bool>
directly into a numpy owned array. We have to copy still since we aren't
deleting the owning PauliProductMeasurement in Rust so we can't use
`into_pyarray()` which would avoid that copy. But this sill reduces the
numbers of copies needed to make the numpy arrays in Python.

Run Details

94214 of 106865 relevant lines covered (88.16%)

1131662.78 hits per line

Qiskit / qiskit / 19169744603 / 1
88%
main: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 19169744603.1

Qiskit / qiskit / 19169744603 / 1 88% main: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 19169744603.1

Qiskit / qiskit / 19169744603 / 1
88%
main: 88%

README BADGES
x