• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

Qiskit / qiskit / 8281628895
89%
main: 88%

Build:
Build:
LAST BUILD BRANCH: substitute_pi4_rotations
DEFAULT BRANCH: main
Ran 14 Mar 2024 01:45PM UTC
Jobs 1
Files 818
Run time 2min
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

14 Mar 2024 01:32PM UTC coverage: 89.253% (-0.05%) from 89.301%
8281628895

push

github

web-flow
Oxidize TwoQubitWeylDecomposition (#11946)

* Oxidize TwoQubitWeylDecomposition

This commit is part 1 of migrating the default 2q unitary synthesis to leverage
parallel rust described in #8774, the eventual goal is to be able to
run unitary synthesis in parallel for all the unitary matrices in a
circuit in a single call from the `UnitarySynthesis` pass. This commit
lays the initial groundwork for doing this by starting with the largest
piece of the default 2q unitary synthesis code, the
TwoQubitWeylDecomposition class. It migrates the entire class
to be a pyclass in rust. There is still a Python subclass for it that
handles the actual QuantumCircuit generation and also the string
representations which are dependent on circuit generation. The goal of
this is to keep the same basic algorithm in place but re-implement
as-is in Rust as a common starting point for eventual improvements to
the underlying algorithm as well as parallelizing the synthesis of
multiple 2q unitary matrices.

* Fix typo in closest_partial_swap()

This commit fixes a typo the formula in the function.
This is the same fix from #11953.

Co-authored-by: Shelly Garion <46566946+ShellyGarion@users.noreply.github.com>

* Run black and lint

* Fix potential imaginary component sign flip in determinant

* Run cargo fmt

* Simplify using numpy eigh example comment

* Add missing checks to decompose_two_qubit_product_gate()

* Use rng first trial from Python implementation

To aid in debugging and rule out rng differences causing different
results this commit switches the first iteration of the randomized loop
to have hard coded values that are identical to what the rng in numpy
was returning. It is very unlikely that this will have any impact
because the specific random numbers used shouldn't matter, this is
mostly to just rule it out as a possibility in debugging the remaining
test failures.

* Fix whitespace in error message

* Fix assert check

* Fix missing multiplier on specialized... (continued)

812 of 897 new or added lines in 4 files covered. (90.52%)

24 existing lines in 5 files now uncovered.

59564 of 66736 relevant lines covered (89.25%)

241706.02 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
20
90.58
-2.28% crates/accelerate/src/euler_one_qubit_decomposer.rs
65
91.4
-1.81% crates/accelerate/src/two_qubit_decompose.rs

Uncovered Existing Lines

Lines Coverage ∆ File
1
97.62
-2.38% qiskit/synthesis/two_qubit/weyl.py
3
95.03
-1.61% qiskit/synthesis/two_qubit/two_qubit_decompose.py
4
91.69
-0.5% crates/qasm2/src/lex.rs
6
96.69
0.0% crates/qasm2/src/parse.rs
10
90.58
-2.28% crates/accelerate/src/euler_one_qubit_decomposer.rs
Jobs
ID Job ID Ran Files Coverage
1 8281628895.1 14 Mar 2024 01:45PM UTC 818
89.25
GitHub Action Run
Source Files on build 8281628895
  • Tree
  • List 818
  • Changed 31
  • Source Changed 0
  • Coverage Changed 7
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • dd802cac on github
  • Prev Build on gh-readonly-queue/main/pr-12001-9f228fa4fe5e8f4de4b53cf77400e25242ba43c3 (#8265059547)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc