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

coverage: 89.831% (-0.009%) from 89.84%
9843343559

push

github

web-flow 
Improve the performance of the `ProductFormula` synthesizers (#12724)

* [WIP] adds the output argument to the internal atomic evolution

* meta: modernize type hints

* refactor: change callable structure of atomic evolution

This changes the structure of the `atomic_evolution` callable in the
`ProductFormula` synthesis class. This is motivated by the significant
performance improvements that can be obtained by appending to the
existing circuit directly rather than building out individual evolution
circuits and iteratively composing them.

* refactor: deprecate the legacy atomic_evolution signature

* refactor: add the wrap argument to ProductFormula

This can be used to recover the previous behavior in which the single
individually evolved Pauli terms get wrapped into gate objects.

* fix: insert the missing barriers between LieTrotter repetitions

* refactor: align SuzukiTrotter and LieTrotter

* Propoagate deprecation notice

* fix: the labels of wrapped Pauli evolutions

* fix: respect the insert_barriers setting

* docs: add a release note

* Apply suggestions from code review

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* fix: missing `wrap` forward in SuzukiTrotter

* docs: improve documentation of the `atomic_evolution` argument

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* docs: also document the deprecated form of `atomic_evolution`

* docs: call out ProductFormula docs in release note

* refactor: change to PendingDeprecationWarning

* refactor: explicitly convert to Gate when wrapping

This is slightly faster than the `.compose`-based operation done
previously as it performs fewer checks. Thanks to @jakelishman for the
suggestion offline.

* Update qiskit/synthesis/evolution/lie_trotter.py

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* docs: update after pending deprecation

---------

Co-authored-by: Julien Gacon <gaconju@gmail.com>

95 of 100 new or added lines in 6 files covered. (95.0%)

13 existing lines in 2 files now uncovered.

65232 of 72616 relevant lines covered (89.83%)

299751.04 hits per line

Relevant lines Covered
72616 RELEVANT LINES 65232 COVERED LINES
299751.04 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-12724-e5533fde0a83f78d889cfc95616688fac969fc70

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
9843343559 gh-readonly-queue/main/pr-12724-e5533fde0a83f78d889cfc95616688fac969fc70 Improve the performance of the `ProductFormula` synthesizers (#12724) * [WIP] adds the output argument to the internal atomic evolution * meta: modernize type hints * refactor: change callable structure of atomic evolution This changes the str... push web-flow github
89.83
See All Builds (9024)