Qiskit / qiskit / 10633814196

89%

main: 88%

Build Type

push

github

Committed by web-flow

Commit Message

Add rust friendly `assign parameters` methods (#12913)

* Initial: Add `assign_parameter` methods that can be used from Rust.

* Add: Error message in `from_slice` method.
- Accept owned mapping for `from_mapping`.

* Fix: Use `assign_parameters_from_slice` as inner method for `assign_parameters_sequence`

* Fix: Use `ParameterUuid` as keys for `assign_parameter_from_mapping`

* Fix: Accept Param references in `assign_parameters_inner`

* Fix: Ownership issues
- Remove initial allocation of `Param` instances within a vec in `assign_parameters_iterable`.
- Revert changes in `assign_parameters_mapping`.
- Fix error message in `assign_parameter_from_slice`.
- Return an error if a `uuid` is not found in `assign_parameters from mapping. Also, return error if a valid uuid is not found to avoid panicking.
- Add `clone_ref(py)` method to `Param` to ensure that we use the most efficient cloning methods (either `clone_ref` with gil or copying).
- Implement trait `AsRef<Param>` for `Param` to be able to send both owned and non-owned instances to `assign_parameters_inner`.

* Docs: Add comment on `AsRef<Param>` impl block.

Run Details

44 of 64 new or added lines in 2 files covered. (68.75%)

39 existing lines in 4 files now uncovered.

71642 of 80355 relevant lines covered (89.16%)

403923.17 hits per line