Qiskit / qiskit / 14931971977

88%

Build Type

push

github

Committed by web-flow

Commit Message

Split out separate qiskit-transpiler crate (#14286)

* Split out separate qiskit-transpiler crate

This commit creates a new qiskit-transpiler crate. As we move towards
exposing more rust functionality directly via the C API having more
explicit interface boundaries in the rust code is important for keeping
things organized and managing dependencies (both external and internal).
We also are building towards a case where the core functionality exists
without a dependency on python, so more explicitly splitting
functionality makes it clearer where the expected boundaries are. The
ideal end state is where we can separate the API of qiskit-transpiler
(and the existing qiskit-circuit) to be pure rust and then move the
python interface components to only live in the accelerate or pyext
crate. But that is a longer term vision and this is only one step
towards that goal.

The transpiler functionality in Rust has grown to the point where it is
both sufficiently self contained and also large enough to warrant its
own compilation unit. Especially because the transpiler is the next big
frontier in the C API, so having it be standalone will make it simpler
to expose via cext.

This PR isn't a 100% clean separation yet though. Ideally
qiskit-transpiler would not depend on qiskit-accelerate. However this PR
still has qiskit-transpiler dependency on qiskit-accelerate. This is for
2 reasons primarily. The first is there are synthesis and quantum info
pieces that are still in qiskit-accelerate which several passes depend.
The next step after this PR is to migrate those into a standalone crate
for synthesis and/or quantum info.  The second component is the nlayout
module. This should live in the transpiler crate, but was not migrated
as part of this PR. This will be done in a follow-up as ideally we
should move PhysicalQubit and VirtualQubit to qiskit-circuit, and the
NLayout struct should move to transpiler. This was deferred as part of
this commit to minimize the chan... (continued)

Run Details

249 of 270 new or added lines in 26 files covered. (92.22%)

2 existing lines in 1 file now uncovered.

76055 of 85723 relevant lines covered (88.72%)

428363.49 hits per line