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/main/pr-12312-d6c74c265ab4d5ed1d16694804d9fd7130003bb5
branch: gh-readonly-queue/main/pr-12312-d6c74c265ab4d5ed1d16694804d9fd7130003bb5
CHANGE BRANCH
x

coverage: 89.523% (+0.01%) from 89.512%
8924853329

push

github

web-flow 
Add equivalence library entry for swap to ECR or CZ (#12312)

* Add equivalence library entry for swap to ECR or CZ

This commit adds two new equivalence library entries to cover the
conversion from a SWAP gate to either ecr or cz directly. These are
common 2q basis gates and without these entries in the equivalence
library the path found from a lookup ends up with a much less efficient
translation. This commit adds the two new entries so that the
BasisTranslator will use a more efficient decomposition from the start
when targeting these basis. This will hopefully result in less work for
the optimization stage as the output will already be optimal and not
require simplification.

Testing for this PR is handled automatically by the built-in testing
harness in test_gate_definitions.py that evaluates all the entries in
the standard equivalence library for unitary equivalence.

* Add name to annotated gate circuit in qpy backwards compat tests

* Fix equivalence library tests

As fallout from the addition of SingletonGate and
SingletonControlledGate we were accidentally not running large portions
of the unit tests which validate the default session equivalence
library. This test dynamically runs based on all members of the standard
gate library by looking at all defined subclasses of Gate and
ControlledGate. But with the introduction of SingletonGate and
SingletonControlledGate all the unparameterized gates in the library
were not being run through the tests. This commit fixes this to catch
that the swap definition added in the previous commit on this PR branch
used an incorrect definition of SwapGate using ECRGate. The definition
will be fixed in a follow up PR.

* Use a more efficient and actually correct circuit for ECR target

The previous definition of a swap gate using ECR rz and sx was incorrect
and also not as efficient as possible. This was missed because the tests
were accidently broken since #10314 which was fixed in the previous
commit. This c... (continued)

24 of 24 new or added lines in 1 file covered. (100.0%)

4 existing lines in 3 files now uncovered.

61703 of 68924 relevant lines covered (89.52%)

216588.53 hits per line

Relevant lines Covered
68924 RELEVANT LINES 61703 COVERED LINES
216588.53 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-12312-d6c74c265ab4d5ed1d16694804d9fd7130003bb5

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
8924853329 gh-readonly-queue/main/pr-12312-d6c74c265ab4d5ed1d16694804d9fd7130003bb5 Add equivalence library entry for swap to ECR or CZ (#12312) * Add equivalence library entry for swap to ECR or CZ This commit adds two new equivalence library entries to cover the conversion from a SWAP gate to either ecr or cz directly. These ... push web-flow github
89.52
See All Builds (9023)