Qiskit / qiskit / 11708174957

88%

Build Type

push

github

Committed by web-flow

Commit Message

Fix: Solve strange behavior with `qubit_properties` in `Target`. (#13400)

The previous implementation of the Target would always make sure to receive a real value for `num_qubits` or it would default to zero. Mistakenly because of the behavior of `Target::from_configuration` it was assumed that `num_qubits` could be `None` unless otherwise specified. However, the default behavior should be to make `num_qubits` zero unless specified otherwise.

The following commits fix this behavior by fixing the signature of the rust `Target` to have default values and adding an extra check when comparing against `qubit_properties`.

A test casde based on @wshanks findings in #13393 was also included in these additions.

Run Details

3 of 3 new or added lines in 1 file covered. (100.0%)

23 existing lines in 4 files now uncovered.

77099 of 86859 relevant lines covered (88.76%)

364275.25 hits per line