Qiskit / qiskit
90%
main: 89%

LAST BUILD BRANCH: remove_deprecated_function_arguments/1
DEFAULT BRANCH: main
Repo Added
Files 914
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH gh-readonly-queue/main/pr-12780-7cd2c414b0663765fe447f5fb91300ca6865f333
branch: gh-readonly-queue/main/pr-12780-7cd2c414b0663765fe447f5fb91300ca6865f333
CHANGE BRANCH
x

coverage: 89.751% (+0.02%) from 89.733%
10144623619

push

github

web-flow 
Add config option to leverage all cores for sabre (#12780)

* Add config option to leverage all cores for sabre

By default when running sabre in parallel we use a fixed number of
threads (depending on optimization level). This was a tradeoff made for
having deterministic results across multiple systems with a fixed seed
set. However when running qiskit on systems with a lot of CPUs
available we're leaving potential performance on the table by not using
all the available cores. This new flag lets users opt-in to running
sabre with n trials for n CPUs to potentially get better output results
from the transpiler, with minimal to no runtime overhead, at the cost of
the results not necessarily being reproducible when run on a different
computer.

* Apply suggestions from code review

Co-authored-by: Elena Peña Tapia <57907331+ElePT@users.noreply.github.com>

* Rework logic to use the default if larger than CPU_COUNT

This commit refactors the logic added in the previous commit to a single
helper function. This reduces the code duplication and makes it easier
to work with. While doing this the logic has been updated so that when
the flag is set and the default number of trials is larger than the
CPU_COUNT we use the default. This means the logic when the flag is set
is to run `max(default_trials, CPU_COUNT)` which should better match
user expectations around the flag.

---------

Co-authored-by: Elena Peña Tapia <57907331+ElePT@users.noreply.github.com>

19 of 22 new or added lines in 2 files covered. (86.36%)

12 existing lines in 3 files now uncovered.

66831 of 74463 relevant lines covered (89.75%)

395720.99 hits per line

Relevant lines Covered
74463 RELEVANT LINES 66831 COVERED LINES
395720.99 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-12780-7cd2c414b0663765fe447f5fb91300ca6865f333

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
10144623619 gh-readonly-queue/main/pr-12780-7cd2c414b0663765fe447f5fb91300ca6865f333 Add config option to leverage all cores for sabre (#12780) * Add config option to leverage all cores for sabre By default when running sabre in parallel we use a fixed number of threads (depending on optimization level). This was a tradeoff made... push web-flow github
89.75
See All Builds (8713)