Qiskit / qiskit / 6910594734

86%

main: 88%

Build Type

push

github

Committed by web-flow

Commit Message

Fix issue with ScheduleBlock qpy serialization and use_symengine (#11261)

* Fix issue with ScheduleBlock qpy serialization and use_symengine

This commit fixes an issue with the use_symengine flag on the qpy.dump()
function. Previously in certain conditions a symengine expression that
was listed in the qpy header as being encoded via symengine was
incorrectly being serialized using sympy. This would cause a failure on
deserialialization as the qpy payload was invalid and symengine could
not parse a symengine representation.

This was originally caught during the development of #10902 as that
switches the default of use_symengine to ``True`` as it makes symengine
a hard requirement. This commit splits it out so the isolated fix can be
backported to 0.45.1.

* Skip symengine test if symengine isn't installed

* Fix skip syntax

* Update releasenotes/notes/fix-schedule-qpy-use-symengine-05ae1dfab73e3ff8.yaml

Co-authored-by: Jake Lishman <jake@binhbar.com>

---------

Co-authored-by: Jake Lishman <jake@binhbar.com>

Run Details

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

6 existing lines in 2 files now uncovered.

65947 of 76759 relevant lines covered (85.91%)

100480.77 hits per line