Qiskit / qiskit / 15437587091

88%

main: 88%

Build Type

push

github

Committed by web-flow

Commit Message

Fix parameter deserialization bugs in `QPY` (#13727)

* Don't assume parameter vector names are unique

* Add from_expr flag

* Add reno

* Specify issue number

Fix black

* Update releasenotes/notes/fix-qpy-fauli-feature-map-33292033755879ee.yaml

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Update fix, add tests

* Update reno

* Update comment

* Update test_circuit_load_from_qpy.py

Co-authored-by: Takashi Imamichi <31178928+t-imamichi@users.noreply.github.com>

* Bump qiskit version, update docs

* Avoid new warning

* Assert user warning is not raised during regression tests (currently fails for legacy ZZFeatureMap)

* Rename flag and add comments

* Remove unnecessary logic

* Document new option

* Document new option

* Apply suggestions from Imamichi-san's code review

Co-authored-by: Takashi Imamichi <31178928+t-imamichi@users.noreply.github.com>

* Update qiskit/qpy/__init__.py

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Cleaner implementation: use parameter vector ROOT uuid as key, resolve element uuids from root uuid

* Improve docs and comments

* Fix lint

* Add parameter equality tests

---------

Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Takashi Imamichi <31178928+t-imamichi@users.noreply.github.com>

Run Details

24 of 24 new or added lines in 3 files covered. (100.0%)

29 existing lines in 3 files now uncovered.

81398 of 92608 relevant lines covered (87.9%)

526278.93 hits per line