update tests to avoid NumPy matrix deprecation

murrayrm · murrayrm · commit f45d61e5f159 · 2021-11-21T13:52:24.000-08:00
diff --git a/control/tests/iosys_test.py b/control/tests/iosys_test.py
@@ -1250,31 +1250,34 @@ def test_lineariosys_statespace(self, tsys):
 
     @pytest.mark.parametrize(
         "Pout, Pin, C, op, PCout, PCin", [
-            (2, 2, ct.rss(2, 2, 2), ct.LinearIOSystem.__mul__, 2, 2),
+            (2, 2, 'rss', ct.LinearIOSystem.__mul__, 2, 2),
             (2, 2, 2, ct.LinearIOSystem.__mul__, 2, 2),
             (2, 3, 2, ct.LinearIOSystem.__mul__, 2, 3),
             (2, 2, np.random.rand(2, 2), ct.LinearIOSystem.__mul__, 2, 2),
-            (2, 2, ct.rss(2, 2, 2), ct.LinearIOSystem.__rmul__, 2, 2),
+            (2, 2, 'rss', ct.LinearIOSystem.__rmul__, 2, 2),
             (2, 2, 2, ct.LinearIOSystem.__rmul__, 2, 2),
             (2, 3, 2, ct.LinearIOSystem.__rmul__, 2, 3),
             (2, 2, np.random.rand(2, 2), ct.LinearIOSystem.__rmul__, 2, 2),
-            (2, 2, ct.rss(2, 2, 2), ct.LinearIOSystem.__add__, 2, 2),
+            (2, 2, 'rss', ct.LinearIOSystem.__add__, 2, 2),
             (2, 2, 2, ct.LinearIOSystem.__add__, 2, 2),
             (2, 2, np.random.rand(2, 2), ct.LinearIOSystem.__add__, 2, 2),
-            (2, 2, ct.rss(2, 2, 2), ct.LinearIOSystem.__radd__, 2, 2),
+            (2, 2, 'rss', ct.LinearIOSystem.__radd__, 2, 2),
             (2, 2, 2, ct.LinearIOSystem.__radd__, 2, 2),
             (2, 2, np.random.rand(2, 2), ct.LinearIOSystem.__radd__, 2, 2),
-            (2, 2, ct.rss(2, 2, 2), ct.LinearIOSystem.__sub__, 2, 2),
+            (2, 2, 'rss', ct.LinearIOSystem.__sub__, 2, 2),
             (2, 2, 2, ct.LinearIOSystem.__sub__, 2, 2),
             (2, 2, np.random.rand(2, 2), ct.LinearIOSystem.__sub__, 2, 2),
-            (2, 2, ct.rss(2, 2, 2), ct.LinearIOSystem.__rsub__, 2, 2),
+            (2, 2, 'rss', ct.LinearIOSystem.__rsub__, 2, 2),
             (2, 2, 2, ct.LinearIOSystem.__rsub__, 2, 2),
             (2, 2, np.random.rand(2, 2), ct.LinearIOSystem.__rsub__, 2, 2),
             
         ])
     def test_operand_conversion(self, Pout, Pin, C, op, PCout, PCin):
         P = ct.LinearIOSystem(
             ct.rss(2, Pout, Pin, strictly_proper=True), name='P')
+        if isinstance(C, str) and C == 'rss':
+            # Need to generate inside class to avoid matrix deprecation error
+            C = ct.rss(2, 2, 2)
         PC = op(P, C)
         assert isinstance(PC, ct.LinearIOSystem)
         assert isinstance(PC, ct.StateSpace)
@@ -1283,20 +1286,24 @@ def test_operand_conversion(self, Pout, Pin, C, op, PCout, PCin):
 
     @pytest.mark.parametrize(
         "Pout, Pin, C, op", [
-            (2, 2, ct.rss(2, 3, 2), ct.LinearIOSystem.__mul__),
-            (2, 2, ct.rss(2, 2, 3), ct.LinearIOSystem.__rmul__),
-            (2, 2, ct.rss(2, 3, 2), ct.LinearIOSystem.__add__),
-            (2, 2, ct.rss(2, 2, 3), ct.LinearIOSystem.__radd__),
+            (2, 2, 'rss32', ct.LinearIOSystem.__mul__),
+            (2, 2, 'rss23', ct.LinearIOSystem.__rmul__),
+            (2, 2, 'rss32', ct.LinearIOSystem.__add__),
+            (2, 2, 'rss23', ct.LinearIOSystem.__radd__),
             (2, 3, 2, ct.LinearIOSystem.__add__),
             (2, 3, 2, ct.LinearIOSystem.__radd__),
-            (2, 2, ct.rss(2, 3, 2), ct.LinearIOSystem.__sub__),
-            (2, 2, ct.rss(2, 2, 3), ct.LinearIOSystem.__rsub__),
+            (2, 2, 'rss32', ct.LinearIOSystem.__sub__),
+            (2, 2, 'rss23', ct.LinearIOSystem.__rsub__),
             (2, 3, 2, ct.LinearIOSystem.__sub__),
             (2, 3, 2, ct.LinearIOSystem.__rsub__),
         ])
     def test_operand_incompatible(self, Pout, Pin, C, op):
         P = ct.LinearIOSystem(
             ct.rss(2, Pout, Pin, strictly_proper=True), name='P')
+        if isinstance(C, str) and C == 'rss32':
+            C = ct.rss(2, 3, 2)
+        elif isinstance(C, str) and C == 'rss23':
+            C = ct.rss(2, 2, 3)
         with pytest.raises(ValueError, match="incompatible"):
             PC = op(P, C)
 
