Qiskit / qiskit
90%
main: 89%

LAST BUILD BRANCH: BD-add-alt-text
DEFAULT BRANCH: main
Repo Added
Files 913
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.2/pr-13255-631abab3493e89fe3df9a77585c395d78b088ac9
branch: gh-readonly-queue/stable/1.2/pr-13255-631abab3493e89fe3df9a77585c395d78b088ac9
CHANGE BRANCH
x

coverage: 89.731% (-0.04%) from 89.768%
11146253635

push

github

web-flow 
Workaround symengine serialization payload incompatibility (backport #13251) (#13255)

* Workaround symengine serialization payload incompatibility (#13251)

* Workaround symengine serialization payload incompatibility

In QPY we rely on symengine's internal serialization to represent the
internal symbolic expression stored inside a ParameterExpression object.
However, this format is nominally symengine version specific and will
raise an error if there is a mismatch between the version used to
generate the payload and what is trying to read it. This became an issue
in the recent symengine 0.13 release which started to raise an error
when people installed it and tried to load QPY payloads across the
versions. This makes the symengine serialization unsuitable for use in
QPY because it's supposed to be independent of these kind of concerns,
especially when QPY is used in a server-client model where you don't
necessarily control the installed environment of symengine.

To correctly address this issue we'll need a new version of the QPY
format that owns the serialization format of ParameterExpressions directly
instead of relying on symengine which doesn't offer a compatibility
guarantee on the format. However this won't be quick solution and users
are encountering issues since the release of 0.13. This commit
introduces a workaround for this specific instance of the mismatch. It
turns out the payload format between 0.11 and 0.13 is completely
unchanged except for the version number. So before passing the parameter
expression payload to symengine for deserialization this commit checks
the versions numbers are the same, if they're not it checks that we're
dealing with 0.11 or 0.13, and if so it changes the version number in
the payloads header appropriately. If the version number is outside
those bounds it raises an exception because while this hack is known to
be safe for translating between symengine 0.11 and 0.13, it's not
possible to know for a future ve... (continued)

11 of 23 new or added lines in 3 files covered. (47.83%)

22 existing lines in 2 files now uncovered.

66873 of 74526 relevant lines covered (89.73%)

219065.95 hits per line

Relevant lines Covered
74526 RELEVANT LINES 66873 COVERED LINES
219065.95 HITS PER LINE
Source Files on gh-readonly-queue/stable/1.2/pr-13255-631abab3493e89fe3df9a77585c395d78b088ac9

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
11146253635 gh-readonly-queue/stable/1.2/pr-13255-631abab3493e89fe3df9a77585c395d78b088ac9 Workaround symengine serialization payload incompatibility (backport #13251) (#13255) * Workaround symengine serialization payload incompatibility (#13251) * Workaround symengine serialization payload incompatibility In QPY we rely on symengine... push web-flow github
89.73
See All Builds (9023)