Qiskit / qiskit / 22686866880

88%

Build Type

push

github

Committed by web-flow

Commit Message

Add VF2PostLayout to the C Transpiler pipeline (#15151)

* Add VF2PostLayout to the C Transpiler pipeline

This commit adds the vf2 post layout transpiler pass to the rust
transpile() function used by qk_transpile() in C. This was not included
originally because there wasn't a path to running vf2postlayout from
rust before but since this has been added now we can run the pass as
part of the C transpilation.

One detail that was added as part of this the function for filling out a
post layout with idle physical qubits was missing from the vf2 post
implementation in rust since for the Python pass it was done Python side
still. This was added as a standalone function for rust since we need it
in rust, but the Python implementation still makes sense for the python
side.

* Use enum for layout source instead of boolean

* Remove reused vf2_config

* Fix lint

* Remove allocate_idle_qubits

After rebasing this and after simplifying the code this function is no
longer needed so we can remove it and rely on the pass code to return a
full layout.

* Fix: Add provisional arguments to C standalone stages
- Since the last time Matthew worked on this PR. All the standalone C API transpiler stages have been added, and they rely on the rust-side stages to function. Since this PR changes the arguments for certain stages. I have added provisional default values that may be removed in future iterations. These are based on the default settings of the transpiler.

* Fix: Add condition when running vf2
- An error in a previous implementation had the VF2 pass run only on optimization level 3 but had enabled configurations for 1 and 2. The confguration and run steps of the pass were moved outisde of this condition.
- VF2 is not used whenever we operate in optimization level 0, so another condition was added to only run the pass if the level is other than zero.

* Fix: Include idle qubits in vf2 mapping.
- The rust version of VF2Post forgets about the idle bits in the mappin... (continued)

Run Details

72 of 81 new or added lines in 3 files covered. (88.89%)

34 existing lines in 5 files now uncovered.

100565 of 114671 relevant lines covered (87.7%)

1149552.51 hits per line