Qiskit / qiskit
89%
main: 89%

LAST BUILD BRANCH: dag-concatenate
DEFAULT BRANCH: main
Repo Added
Files 896
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/main/pr-13358-f2e07bc52640c0583c4ee62f9e5b8ab30460a03d
branch: gh-readonly-queue/main/pr-13358-f2e07bc52640c0583c4ee62f9e5b8ab30460a03d
CHANGE BRANCH
x

coverage: 88.675% (+0.001%) from 88.674%
11517016655

push

github

web-flow 
Fix compatibility issues with SciPy 1.14 (#13358)

* Fix compatibility issues with SciPy 1.14

The main change here is that SciPy 1.14 on macOS now uses the system
Accelerate rather than a bundled OpenBLAS by default.  These have
different characteristics for several LAPACK drivers, which caused
numerical instability in our test suite.  Fundamentally, these problems
existed before; it was always possible to switch out the BLAS/LAPACK
implementation that SciPy used, but in practice, the vast majority of
users (and our CI) use the system defaults.

The modification to `Operator.power` to shift the branch cut was
suggested by Lev.  As a side-effect of how it's implemented, it fixes
an issue with `Operator.power` on non-unitary matrices, which Sasha had
been looking at.

The test changes to the Kraus and Stinespring modules are to cope with
the two operators only being defined up to a global phase, which the
test previously did not account for.  The conversion to Kraus-operator
form happens to work fine with OpenBLAS, but caused global-phase
differences on macOS Accelerate.  A previous version of this commit
attempted to revert the Choi-to-Kraus conversion back to using `eigh`
instead of the Schur decomposition, but the `eigh` instabilities noted
in fdd5603af76 (gh-3884) (the time of Scipy 1.1 to 1.3, with OpenBLASes
around 0.3.6) remain with Scipy 1.13/1.14 and OpenBLAS 0.3.27.

Co-authored-by: Lev S. Bishop <18673315+levbishop@users.noreply.github.com>
Co-authored-by: Alexander Ivrii <alexi@il.ibm.com>

* Expose `branch_cut_rotation` parameter in `Operator.power`

The rotation used to stabilise matrix roots has an impact on which
matrix is selected as the principal root.  Exposing it to users to allow
control makes the most sense.

---------

Co-authored-by: Lev S. Bishop <18673315+levbishop@users.noreply.github.com>
Co-authored-by: Alexander Ivrii <alexi@il.ibm.com>

10 of 10 new or added lines in 2 files covered. (100.0%)

17 existing lines in 3 files now uncovered.

74895 of 84460 relevant lines covered (88.68%)

366470.02 hits per line

Relevant lines Covered
84460 RELEVANT LINES 74895 COVERED LINES
366470.02 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-13358-f2e07bc52640c0583c4ee62f9e5b8ab30460a03d

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
11517016655 gh-readonly-queue/main/pr-13358-f2e07bc52640c0583c4ee62f9e5b8ab30460a03d Fix compatibility issues with SciPy 1.14 (#13358) * Fix compatibility issues with SciPy 1.14 The main change here is that SciPy 1.14 on macOS now uses the system Accelerate rather than a bundled OpenBLAS by default. These have different charact... push web-flow github
88.68
See All Builds (8468)