Qiskit / qiskit / 15302797831

Build # 15302797831

Build Type

push

github

Committed by web-flow

Commit Message

Avoid all-to-all connectivity in slowest test (#14483) (#14485)

The current slowest unit test in the suite is:

test_transpile_does_not_affect_backend_coupling_4_3

which is just sanity checking that a transpile() call doesn't mutate the
CouplingMap by mistake. This is is a regression test for an old bug where
that happened. However, in the move to use GenericBackendV2 it was
creating a 130 qubit target with explicit all-to-all connectivity, which
is the default behavior if you don't specify a coupling map argument.
This ends up bogging the transpiler down in VF2PostLayout in
optimization level 3 because there are a huge number of possible
layouts, basically any permutation of 130 qubits. This isn't
functionally part of the test because it's just trying to test if the
coupling map is mutated by the transpiler. This commit updates the test
to use a 130 qubit linear connectivity graph which still tests the
circuit but takes a fraction of the time to execute because the target
is much more constrained.

(cherry picked from commit 88ce515da)

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

Run Details

72920 of 82762 relevant lines covered (88.11%)

347622.02 hits per line

Source File

Press 'n' to go to next uncovered line, 'b' for previous

Source Not Available