From: Michael Orlitzky <michael@orlitzky.com>
Date: Wed, 9 Nov 2016 14:36:39 +0000 (-0500)
Subject: Remove the "math domain error" TODO, addressed by twiddling MAX_COND.
X-Git-Tag: 0.1.0~57
X-Git-Url: https://gitweb.michael.orlitzky.com/?a=commitdiff_plain;h=cb004dc613ea1bc9596e7d4856c16f7cbe070772;p=dunshire.git

Remove the "math domain error" TODO, addressed by twiddling MAX_COND.
---

diff --git a/TODO b/TODO
index 3febfcd..d75812b 100644
--- a/TODO
+++ b/TODO
@@ -79,31 +79,6 @@
       [0.0000000]
 
 
-6. Fix the math domain errors that sometimes pop up:
-
-  ERROR: test_scaling_icecream (test.symmetric_linear_game_test
-                                .SymmetricLinearGameTest)
-  ----------------------------------------------------------------------
-  Traceback (most recent call last):
-    File "/home/mjo/src/dunshire/test/symmetric_linear_game_test.py",
-    line 336, in test_scaling_icecream
-      self.assert_scaling_works(L, K, e1, e2)
-    File "/home/mjo/src/dunshire/test/symmetric_linear_game_test.py",
-    line 317, in assert_scaling_works
-      value2 = game2.solution().game_value()
-    File "/home/mjo/src/dunshire/dunshire/games.py", line 428, in solution
-      soln_dict = solvers.conelp(c, G, h, C.cvxopt_dims(), A, b)
-    File "/usr/lib64/python3.4/site-packages/cvxopt/coneprog.py", line 1395,
-    in conelp
-      misc.update_scaling(W, lmbda, ds, dz)
-    File "/usr/lib64/python3.4/site-packages/cvxopt/misc.py", line 510,
-    in update_scaling
-      ln = jnrm2(lmbda, n = m, offset = ind)
-    File "/usr/lib64/python3.4/site-packages/cvxopt/misc.py", line 856, in jnrm2
-      return math.sqrt(x[offset] - a) * math.sqrt(x[offset] + a)
-  ValueError: math domain error
-
-
 7. Figure out why this happens, too:
 
   FAIL: test_scaling_icecream (test.symmetric_linear_game_test