From 744bc420b47449500f0365e093467ffd268aeda2 Mon Sep 17 00:00:00 2001 From: Michael Orlitzky Date: Fri, 8 Jan 2016 00:53:34 -0500 Subject: [PATCH] Give an example where check=True is necessary for the pi/Z cones. --- mjo/cone/cone.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/mjo/cone/cone.py b/mjo/cone/cone.py index d62ffa2..8790c30 100644 --- 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... -- 2.43.2