Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: gh-readonly-queue/main/pr-15094-96f5c08b3f54acfcc9552a34bf1bcc5402e07e7f
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-14381-d522aa4c0781ff447f7dad424a58a6dc800e87d8
branch: gh-readonly-queue/main/pr-14381-d522aa4c0781ff447f7dad424a58a6dc800e87d8
CHANGE BRANCH
x

coverage: 88.294% (-0.03%) from 88.321%
15078856691

push

github

web-flow 
Revert "Port star_preroute to rust" (#14381)

* Revert "Port star_preroute to rust"

This commit reverts the move of star prerouting to run in Rust. At the
time of this commit it was before the majority of the transpiler
infrastructure was ported to rust. At that time the pass was ported we
had a few one off passes that were ported to rust and we had to manage
the Python rust boundary carefully. The pattern was typically to define
custom rust data types that provided an alternate view of the
DAGCircuit and the target for compilation that was built out of the
Python space dag and target. The star prerouting pass was ported to
Rust and leveraged the existing infrastructure used by SabreSwap and
SabreLayout as the function of the pass on a DAG was very similar to
Sabre (finding where to insert swaps and rebuilding the dag where
swaps are needed). At the time this was a pragmatic choice to
accelerate the pass and not need to wait for the needed infrastructure
to be written in Rust. However in practice the performance gains the
Rust porting of the algorithm provided were not as large as was
originally hoped and now the code of the pass is tied directly to
the current implementation of sabre. As the sabre interface layer is
being re-written in #14317 this is proving to be a blocker for that
effort. This commit reverts the current rust implementation to unblock
the sabre refactoring effort.

It's still definitely worth porting this pass to Rust but we now have
sufficient infrastructure in Rust to write full transpiler passes
without Python. We should follow up this PR and rewrite the star
prerouting pass so that full pass operates completely in rust and
directly leverages the native data structures that exist in Rust now
instead of the hybrid approach that ties the is tightly coupled to the
internals of the sabre pass.

This reverts commit b23c54523.

* Add back reverted alt text for plot in docs

49 of 50 new or added lines in 1 file covered. (98.0%)

24 existing lines in 3 files now uncovered.

78326 of 88710 relevant lines covered (88.29%)

427402.33 hits per line

Relevant lines Covered
88710 RELEVANT LINES 78326 COVERED LINES
427402.33 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14381-d522aa4c0781ff447f7dad424a58a6dc800e87d8

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
15078856691 gh-readonly-queue/main/pr-14381-d522aa4c0781ff447f7dad424a58a6dc800e87d8 Revert "Port star_preroute to rust" (#14381) * Revert "Port star_preroute to rust" This commit reverts the move of star prerouting to run in Rust. At the time of this commit it was before the majority of the transpiler infrastructure was ported ... push web-flow github
88.29
See All Builds (13861)