1
87%
main: 88%

Ran 02 Mar 2025 12:32AM UTC

Files 867

Run time 20s

Badge

Embed ▾

Committed 02 Mar 2025 12:11AM UTC coverage: 86.965% (-0.02%) from 86.98%

Job # 13610146568.1

Build Type

push

github

Committed by

web-flow

Commit Message

Fix qpy serialization of substitution of type `ParameterExpression` (#13890)

* Fix qpy serialization of substitution of type ParameterExpression

When substitution history `ParameterExpression._qpy_replay` is serialized,
there was no branch for the case that the substituted value is of type
`ParameterExpression`.
This commit fixes this oversight.

* Fix deserializing qpy written by previous writing fix

* Added a branch for reading `ParameterExpression` in _qpy_replay

* Added a missing argument (version) in an existing call in code path that
  was previously untested

* Remove a bit of useless code introduced in last commit

* run black

* Add tests

* Revert renaming extra_symbols to extra_expressions

Reverting this to minimize changes in order to make the PR safer to
backport.

* Revert cleaning up logic in _encode_replay_subs

This was a good change. But not necessary for the bug fix, which is
the main point of this PR.

* Add qpy compat test

* Bind parameter when checking for equality

* Run black after merge in browser

---------

Co-authored-by: Luciano Bello <bel@zurich.ibm.com>

Run Details

76077 of 87480 relevant lines covered (86.97%)

332774.63 hits per line

Qiskit / qiskit / 13610146568 / 1
87%
main: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 13610146568.1

Qiskit / qiskit / 13610146568 / 1 87% main: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 13610146568.1

Qiskit / qiskit / 13610146568 / 1
87%
main: 88%

README BADGES
x