From 8bed8a3c21a0d19d01a7625b3849ac07c9a9f9e6 Mon Sep 17 00:00:00 2001 From: Michael Orlitzky Date: Sun, 11 Nov 2018 22:11:53 -0500 Subject: [PATCH] cone/rearrangement.py: fix the test for propriety. We had a test for the rearrangement cone being proper, but it incorrectly tested the p=0 case. Now that we're sanity-checking p, that test was throwing a ValueError. Fix it to use p >= 1. --- mjo/cone/rearrangement.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mjo/cone/rearrangement.py b/mjo/cone/rearrangement.py index 76b0dad..fd792c4 100644 --- a/mjo/cone/rearrangement.py +++ b/mjo/cone/rearrangement.py @@ -69,7 +69,7 @@ def rearrangement_cone(p,n): sage: all( rearrangement_cone(p,n).is_proper() ....: for n in xrange(10) - ....: for p in xrange(n) ) + ....: for p in xrange(1, n) ) True The Lyapunov rank of the rearrangement cone of order ``p`` in ``n`` -- 2.43.2