Qiskit / qiskit
89%
main: 89%

LAST BUILD BRANCH: dag-concatenate
DEFAULT BRANCH: main
Repo Added
Files 896
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-13375-e7fc5a73ede95acfd3c47f7eba37c47069e24599
branch: gh-readonly-queue/main/pr-13375-e7fc5a73ede95acfd3c47f7eba37c47069e24599
CHANGE BRANCH
x

coverage: 88.656% (-0.03%) from 88.685%
11577253570

push

github

web-flow 
Fix performance regression in UnitarySynthesis (#13375)

* Fix performance regression in UnitarySynthesis

This commit fixes a performance regression that was introduced in
PR #13141. When the pass is looking up the preferred synthesis direction
for a unitary based on the connectvity constraints the connectivity was
being provided as a PyList. To look up the edge in connectivity set this
meant we needed to iterate over the list and then create a set that rust
could lookup if it contains an edge or it's reverse. This has
significant overhead because its iterating via python and also iterating
per decomposition. This commit addresses this by changing the input type
to be a HashSet from Python so Pyo3 will convert a pyset directly to a
HashSet once at call time and that's used by reference for lookups
directly instead of needing to iterate over the list each time.

* Avoid constructing plugin data views with default plugin

If we're using the default plugin the execution will happen in rust
and we don't need to build the plugin data views that are defined in the
plugin interface. Profiling the benchpress test using the hamlib
hamiltonian:

ham_graph-2D-grid-nonpbc-qubitnodes_Lx-5_Ly-186_h-0.5-all-to-all

that caught this originally regression was showing an inordinate amount
of time being spent in the construction of
`_build_gate_lengths_by_qubit` and `_build_gate_errors_by_qubit` which
isn't being used because this happens internally in rust now. To
mitigate this overhead this commit migrates the sections computing these
values to the code branch that uses them and not the default plugin
branch that uses rust.

31 of 32 new or added lines in 2 files covered. (96.88%)

33 existing lines in 4 files now uncovered.

74971 of 84564 relevant lines covered (88.66%)

364508.59 hits per line

Relevant lines Covered
84564 RELEVANT LINES 74971 COVERED LINES
364508.59 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-13375-e7fc5a73ede95acfd3c47f7eba37c47069e24599

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
11577253570 gh-readonly-queue/main/pr-13375-e7fc5a73ede95acfd3c47f7eba37c47069e24599 Fix performance regression in UnitarySynthesis (#13375) * Fix performance regression in UnitarySynthesis This commit fixes a performance regression that was introduced in PR #13141. When the pass is looking up the preferred synthesis direction f... push web-flow github
88.66
See All Builds (8468)