Qiskit / qiskit / 11505520943

88%

Build Type

push

github

Committed by web-flow

Commit Message

Remove stale `variable_class_operations` set from `Target` (#12957)

* Fix: Return None for variadic properties.

* Fix: Remove `variable_class_operations` from `Target`.
- When performing serialization, we were forgetting to include `variable_class_operations` set of names in the state mapping. Since the nature of `TargetOperation` is to work as an enum of either `Instruction` instances or class aliases that would represent `Variadic` instructiions. The usage of that structure was redundand, so it was removed.
- `num_qubits` returns an instance of `u32`, callers will need to make sure they're dealing with a `NormalOperation`.
- `params` behaves more similarly, returning a slice of `Param` instances. Will panic if called on a `Variadic` operation.
- Re-adapt the code to work without `variable_class_operations`.
- Add test case to check for something similar to what was mentioned by @doichanj in #12953.

* Fix: Use `UnitaryGate` as the example in test-case.
- Move import of `pickle` to top of the file.

Run Details

22 of 28 new or added lines in 1 file covered. (78.57%)

17 existing lines in 4 files now uncovered.

74900 of 84467 relevant lines covered (88.67%)

364240.39 hits per line