Qiskit / qiskit
89%
main: 88%

LAST BUILD BRANCH: fix-consolidate-blocks-recursion
DEFAULT BRANCH: main
Repo Added
Files 868
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-13759-4d2265f62a56d23e95bcc273705b048e3ed95acc
branch: gh-readonly-queue/main/pr-13759-4d2265f62a56d23e95bcc273705b048e3ed95acc
CHANGE BRANCH
x

coverage: 88.618% (-0.03%) from 88.651%
13167960323

push

github

web-flow 
Add unitary gate representation to rust (#13759)

* Add unitary gate representation to rust

This commit expands the rust circuit data model to include a definition
of UnitaryGate. This is a more abstract operation than what we've
defined so far in Rust, a gate represented solely by it's unitary matrix
but during normal transpilation we create and interact with these
operations for optimization purposes so having a native representation
in rust is necessary to avoid calling to Python when working with them.

This introduces a new UnitaryGate struct which represents the unpacked
operation. It has 3 internal storage variants based on either an ndarray
arbitrary sized array, a 2x2 nalgebra fixed sized array, or a 4x4
nalgebra fixed sized array. From the python perspective these all look
the same, but being able to internally work with all 3 variants lets us
optimize the code paths for 1q and 2q unitary gates (which are by far
more common) and avoid one layer of pointer indirection.

When stored in a circuit the packed representation is just a pointer to
the actual UnitaryGate which we put in a Box. This is necessary because
the struct size is too large to fit in our compact representation of
operations. Even the ndarray which is a heap allocated type requires more
than our allotted space in a PackedOperation so we need to reduce it's
size by putting it in a Box.

The one major difference here from the previous python based
representation the unitary matrix was stored as an object type parameter
in the PackedInstruction.params field, which now it is stored internally
in the operation itself. There is arguably a behavior change around
this because it's no longer possible to mutate the array of a
UnitaryGate in place once it's inserted into the circuit. While doing
this was horribly unsound, because there was no guardrails for doing it
a release note is added because there is a small use case where it would
have worked and it wasn't explicitly documented.

... (continued)

117 of 177 new or added lines in 6 files covered. (66.1%)

6 existing lines in 4 files now uncovered.

79353 of 89545 relevant lines covered (88.62%)

349522.47 hits per line

Relevant lines Covered
89545 RELEVANT LINES 79353 COVERED LINES
349522.47 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-13759-4d2265f62a56d23e95bcc273705b048e3ed95acc

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
13167960323 gh-readonly-queue/main/pr-13759-4d2265f62a56d23e95bcc273705b048e3ed95acc Add unitary gate representation to rust (#13759) * Add unitary gate representation to rust This commit expands the rust circuit data model to include a definition of UnitaryGate. This is a more abstract operation than what we've defined so far i... push web-flow github
88.62
See All Builds (13862)