From: Michael Orlitzky <michael@orlitzky.com>
Date: Mon, 12 Nov 2018 03:11:53 +0000 (-0500)
Subject: cone/rearrangement.py: fix the test for propriety.
X-Git-Url: http://gitweb.michael.orlitzky.com/?a=commitdiff_plain;h=8bed8a3c21a0d19d01a7625b3849ac07c9a9f9e6;p=sage.d.git

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.
---

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``