Skip to content

Commit

Permalink
Pin cvxpy to known-good version (#10997)
Browse files Browse the repository at this point in the history
Some diamond-norm tests are failing with the latest 1.4.0 release of
cvxpy.  Given the complex optimisation problem that the dnorm
represents, it's quite possible that randomised tests would already have
had a low success rate, and this is just a case of some internal tweaks
destabilising the particular operator choice in our test.

(cherry picked from commit 337832e)
  • Loading branch information
jakelishman authored and mergify[bot] committed Oct 10, 2023
1 parent fd6dc8a commit f087a17
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,9 @@ numpy<1.25
# eigensystem code for one of the test cases. See
# https://github.com/Qiskit/qiskit-terra/issues/10345 for current details.
scipy<1.11

# cvxpy 1.4.0 caused an instability in some diamond-norm tests. That's
# likely to be an unreliable measure already given its nature, so this
# pin may be relaxed by modifying the tests slightly to choose known-safe
# operators.
cvxpy==1.3.2

0 comments on commit f087a17

Please sign in to comment.