1
88%
main: 88%

Ran 28 Oct 2025 07:28PM UTC

Files 873

Run time 39s

Badge

Embed ▾

Committed 28 Oct 2025 06:23PM UTC coverage: 88.182% (-0.008%) from 88.19%

Job # 18885771704.1

Build Type

push

github

Committed by

web-flow

Commit Message

Fix reuse of `ConsolidateBlocks` instances (#15258)

* Fix reuse of `ConsolidateBlocks` instances

The `_qubit_map` state added to `ConsolidateBlocks` in #15083 (cb103d95)
was attempting to get round an awkward internal `PassManager` use to
pass the mapping into recursive calls.  Unfortunately, the state was not
cleared on exit to the pass, so re-use of the pass object would be
invalid, such as by calling the `PassManager.run` method on the result
of `generate_preset_pass_manager` more than once on circuits with
different sizes.

Rather than trying to plaster over the poor use of state, this commit
instead moves the runtime logic to be in the `PropertySet`, so re-use
troubles are limited to single executions.  It's still technically
possible for the `PropertySet` to become polluted if `ConsolidateBlocks`
throws an internal exception that is forcibly suppressed such that the
pipeline continues, but this is completely non-standard, would require
significant effort to achieve, and most likely there are large numbers
of transpiler passes that are not completely exception-safe anyway.

* Add test of `PassManager.run` over a list

On some platforms this will be multiprocessing, but it doesn't hurt to
try it for the platforms that aren't.

* Document use of property-set key

Run Details

93655 of 106206 relevant lines covered (88.18%)

1151764.64 hits per line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 18885771704.1

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 18885771704.1

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

README BADGES
x