Qiskit / qiskit / 13553732677

88%

Build Type

push

github

Committed by web-flow

Commit Message

Refactor parallelism utilities for public API (#12412)

* Refactor parallelism utilities for public API

`should_run_in_parallel` was added in a stable manner to enable backport
to the 1.1 series, but from 2.0 onwards, we want this to be part of the
public interface so that others can rely on it too.

As part of this, the parallelisation configuration was made more robust
and controllable with context managers.  This is convenient beyond just
for users - it makes it far easier to control the parallelism during the
test suite runs.  Several instances where different parts of Qiskit and
its test suite reached into deep internals of the parallelism utilities
and made significant assumptions about the internal logic are refactored
to use public interfaces to achieve what they wanted to.

The multiprocessing detection is changed from making OS-based
assumptions about what Python does to simply querying the module for its
configuration.  This makes it more robust to changes in Python's
handling (especially important since 3.14 will change the default start
method on Unix).  In the future, we may want to change to making these
assumptions only if the user hasn't configured the `multiprocessing`
start method themselves.

* Dodge use of `print`

* Fix typo

Co-authored-by: Kevin Hartman <kevin@hart.mn>

* Gracefully handle bad user input

---------

Co-authored-by: Kevin Hartman <kevin@hart.mn>
Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

Run Details

73 of 98 new or added lines in 8 files covered. (74.49%)

15 existing lines in 5 files now uncovered.

77531 of 88244 relevant lines covered (87.86%)

341893.83 hits per line