Qiskit / qiskit
90%
main: 89%

LAST BUILD BRANCH: unitary_synth
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-12197-95476b747163479fca5ad2ec81b4b3e9fd7fd657
branch: gh-readonly-queue/main/pr-12197-95476b747163479fca5ad2ec81b4b3e9fd7fd657
CHANGE BRANCH
x

coverage: 89.516% (+0.03%) from 89.488%
8901854210

push

github

web-flow 
Oxidize the numeric code in the Isometry gate class (#12197)

* Oxidize the numeric code in the Isometry gate class

This commit ports the numeric portion of the Isometry gate class to
rust. While this will likely improve the performance slightly this move
is more to make isolate this code from blas/lapack in numpy. We're
hitting some stability issues on arm64 mac in CI and moving this code to
rust should hopefully fix this issue. As this is more for functional
reasons no real performance tuning was done on this port, there are
likely several opportunities to improve the runtime performance of the
code.

* Remove unused import

* Use rust impl for small utility functions too

* Oxidize the linalg in UCGate too

The UCGate class is used almost exclusively by the Isometry class to
build up the definition of the isometry circuit. There were also some
linear algebra inside the function which could also be the source of the
stability issues we were seeing on arm64. This commit ports this
function as part of the larger isometry migration.

* Migrate another numeric helper method of UCGate

* Remove now unused code paths

* Remove bitstring usage with bitwise ops

This commit removes the use of bit string manipulations that were
faithfully ported from the original python logic (but left a bad taste
in my mouth) into more efficient bitwise operations (which were
possible in the original python too).

* Mostly replace Vec<u8> usage with bitwise operations

The use of intermediate Vec<u8> as proxy bitstrings was originally
ported nearly exactly from the python implementation. But since
everything is working now this commit switches to use bitwise operations
where it makes sense as this will be more efficient.

* Apply suggestions from code review

Co-authored-by: Jake Lishman <jake@binhbar.com>

* Remove python side call sites

* Fix integer typing in uc_gate.rs

* Simplify basis state bitshift loop logic

* Build set of control labels outside construct_basis_s... (continued)

427 of 428 new or added lines in 6 files covered. (99.77%)

13 existing lines in 2 files now uncovered.

61372 of 68560 relevant lines covered (89.52%)

216392.02 hits per line

Relevant lines Covered
68560 RELEVANT LINES 61372 COVERED LINES
216392.02 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-12197-95476b747163479fca5ad2ec81b4b3e9fd7fd657

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
8901854210 gh-readonly-queue/main/pr-12197-95476b747163479fca5ad2ec81b4b3e9fd7fd657 Oxidize the numeric code in the Isometry gate class (#12197) * Oxidize the numeric code in the Isometry gate class This commit ports the numeric portion of the Isometry gate class to rust. While this will likely improve the performance slightly ... push web-flow github
89.52
See All Builds (9024)