Give an example where check=True is necessary for the pi/Z cones.

diff --git a/mjo/cone/cone.py b/mjo/cone/cone.py
index d62ffa22a70151ba2044b5e52c136604805e1d82..8790c30673a25af96c29bf22ff9e84458516da09 100644 (file)
--- a/mjo/cone/cone.py
+++ b/mjo/cone/cone.py
@@ -421,8 +421,9 @@ def positive_operator_gens(K):
     vectors = [ W(tp.list()) for tp in tensor_products ]
 
     # Create the *dual* cone of the positive operators, expressed as
-    # long vectors. WARNING: takes forever unless we pass check=False
-    # to Cone().
+    # long vectors. WARNING: check=True is necessary even though it
+    # makes Cone() take forever. For an example take
+    # K = Cone([(1,0,0),(0,0,1),(0,0,-1)]).
     pi_dual = Cone(vectors, ToricLattice(W.dimension()))
 
     # Now compute the desired cone from its dual...
@@ -538,8 +539,9 @@ def Z_transformation_gens(K):
     vectors = [ W(m.list()) for m in tensor_products ]
 
     # Create the *dual* cone of the cross-positive operators,
-    # expressed as long vectors. WARNING: takes forever unless we pass
-    # check=False to Cone().
+    # expressed as long vectors. WARNING: check=True is necessary
+    # even though it makes Cone() take forever. For an example take
+    # K = Cone([(1,0,0),(0,0,1),(0,0,-1)]).
     Sigma_dual = Cone(vectors, lattice=ToricLattice(W.dimension()))
 
     # Now compute the desired cone from its dual...