Qiskit / qiskit
90%
main: 89%

LAST BUILD BRANCH: BD-add-alt-text
DEFAULT BRANCH: main
Repo Added
Files 913
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/stable/1.2/pr-12920-3e88060a49188e6b2439aef2a1b5523e7cd47755
branch: gh-readonly-queue/stable/1.2/pr-12920-3e88060a49188e6b2439aef2a1b5523e7cd47755
CHANGE BRANCH
x

coverage: 89.951% (-0.003%) from 89.954%
10295095033

push

github

web-flow 
Remove init peephole optimization discrete basis check (#12898) (#12920)

* Fix target handling in discrete basis check

In #12727 a check was added to the default init stage's construction to
avoid running 2q gate consolidation in the presence of targets with
only discrete gates. However the way the target was being used in this
check was incorrect. The name for an instruction in the target should be
used as its identifier and then if we need the object representation
that should query the target for that object based on the name. However
the check was doing this backwards getting a list of operation objects
and then using the name contained in the object. This will cause issues
for instructions that use custom names such as when there are tuned
variants or a custom gate instance with a unique name.

While there is some question over whether we need this check as we will
run the consolidate 2q blocks pass as part of the optimization stage
which will have the same effect, this opts to just fix the target usage
for it to minimize the diff. Also while not the explicit goal of this
check it is protecting against some bugs in the consolidate blocks pass
that occur when custom gates are used. So for the short term this check
is retained, but in the future when these bugs in consolidate blocks are
fixed we can revisit whether we want to remove the conditional logic.

* Remove check and fix ConsolidateBlocks bug

This commit pivots this PR branch to just remove the additional logic
around skipping the optimization passes for discrete basis sets. The
value the check was actually providing was not around a discrete basis
set target and instead was to workaround a bug in the consolidate blocks
pass. If a discrete basis set target was used this would still fail
because we will unconditionally call `ConsolidateBlocks` during the
optimization stage. This commit opts to just remove the extra complexity
of the conditional execution of the peephole optimization passe... (continued)

9 of 10 new or added lines in 2 files covered. (90.0%)

4 existing lines in 1 file now uncovered.

66720 of 74174 relevant lines covered (89.95%)

232708.25 hits per line

Relevant lines Covered
74174 RELEVANT LINES 66720 COVERED LINES
232708.25 HITS PER LINE
Source Files on gh-readonly-queue/stable/1.2/pr-12920-3e88060a49188e6b2439aef2a1b5523e7cd47755

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
10295095033 gh-readonly-queue/stable/1.2/pr-12920-3e88060a49188e6b2439aef2a1b5523e7cd47755 Remove init peephole optimization discrete basis check (#12898) (#12920) * Fix target handling in discrete basis check In #12727 a check was added to the default init stage's construction to avoid running 2q gate consolidation in the presence of... push web-flow github
89.95
See All Builds (9023)