Qiskit / qiskit
90%
main: 88%

Repo Added 21 Aug 2023 02:51PM UTC

Committed 08 Aug 2024 01:01AM UTC coverage: 89.96% (+0.02%) from 89.938%

Committed by

Commit Message

Remove init peephole optimization discrete basis check (#12898)

* 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 passes and
ins... (continued)

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

4 existing lines in 2 files now uncovered.

66726 of 74173 relevant lines covered (89.96%)

233418.62 hits per line

74173 RELEVANT LINES 66726 COVERED LINES

233418.62 HITS PER LINE

Builds	Branch	Commit	Type	Ran	Committer	Via	Coverage
10294102649	mergify/bp/stable/1.2/pr-12898	Remove init peephole optimization discrete basis check (#12898) * 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 ...	Pull #12920	08 Aug 2024 01:22AM UTC	mergify[bot]	github	89.96

Qiskit / qiskit
90%
main: 88%

Qiskit / qiskit 90% main: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

CHANGE BRANCH x

Relevant lines Covered

Source Files on mergify/bp/stable/1.2/pr-12898

Recent builds

Qiskit / qiskit
90%
main: 88%

README BADGES
x

CHANGE BRANCH
x