Qiskit / qiskit
89%
main: 88%

LAST BUILD BRANCH: gh-readonly-queue/main/pr-15094-96f5c08b3f54acfcc9552a34bf1bcc5402e07e7f
DEFAULT BRANCH: main
Repo Added
Files 868
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH gh-readonly-queue/stable/1.4/pr-13943-06ac96e84cda7f831f6fad78a1782d7c84ed919d
branch: gh-readonly-queue/stable/1.4/pr-13943-06ac96e84cda7f831f6fad78a1782d7c84ed919d
CHANGE BRANCH
x

coverage: 88.862% (+0.02%) from 88.846%
13636111155

push

github

web-flow 
Fix qpy serialization of substitution of type `ParameterExpression` (backport #13890) (#13943)

* 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>
(cherry picked from commit a6fa6f87d)

# Conflicts:
#	test/qpy_compat/test_qpy.py

* Update test_qpy.py

* Update test/qpy_compat/test_qpy.py

---------

Co-authored-by: John Lapeyre <jlapeyre@users.noreply.github.com>
Co-authored-by: Elena Peña Tapia <57907331+ElePT@users.noreply.github.com>
Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

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

7 existing lines in 1 file now uncovered.

79400 of 89352 relevant lines covered (88.86%)

358592.46 hits per line

Relevant lines Covered
89352 RELEVANT LINES 79400 COVERED LINES
358592.46 HITS PER LINE
Source Files on gh-readonly-queue/stable/1.4/pr-13943-06ac96e84cda7f831f6fad78a1782d7c84ed919d

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
13636111155 gh-readonly-queue/stable/1.4/pr-13943-06ac96e84cda7f831f6fad78a1782d7c84ed919d Fix qpy serialization of substitution of type `ParameterExpression` (backport #13890) (#13943) * Fix qpy serialization of substitution of type `ParameterExpression` (#13890) * Fix qpy serialization of substitution of type ParameterExpression Wh... push web-flow github
88.86
See All Builds (13861)