changed error due to non-symmetric array to be ValueError instead of ControlDimension

sawyerbfuller · sawyerbfuller · commit 82d328a869b3 · 2021-11-09T14:49:53.000-08:00
diff --git a/control/mateqn.py b/control/mateqn.py
@@ -163,7 +163,7 @@ def lyap(A, Q, C=None, E=None):
             raise ControlDimension("Q must be a square matrix.")
 
         if not _is_symmetric(Q):
-            raise ControlDimension("Q must be a symmetric matrix.")
+            raise ValueError("Q must be a symmetric matrix.")
 
         # Solve the Lyapunov equation by calling Slycot function sb03md
         try:
@@ -345,7 +345,7 @@ def dlyap(A, Q, C=None, E=None):
             raise ControlDimension("Q must be a square matrix.")
 
         if not _is_symmetric(Q):
-            raise ControlDimension("Q must be a symmetric matrix.")
+            raise ValueError("Q must be a symmetric matrix.")
 
         # Solve the Lyapunov equation by calling the Slycot function sb03md
         try:
@@ -408,7 +408,7 @@ def dlyap(A, Q, C=None, E=None):
                 dimension as A.")
 
         if not _is_symmetric(Q):
-            raise ControlDimension("Q must be a symmetric matrix.")
+            raise ValueError("Q must be a symmetric matrix.")
 
         # Solve the generalized Lyapunov equation by calling Slycot
         # function sg03ad
@@ -585,10 +585,10 @@ def care_slycot(A, B, Q, R=None, S=None, E=None, stabilizing=True):
             raise ControlDimension("Incompatible dimensions of B matrix.")
 
         if not _is_symmetric(Q):
-            raise ControlDimension("Q must be a symmetric matrix.")
+            raise ValueError("Q must be a symmetric matrix.")
 
         if not _is_symmetric(R):
-            raise ControlDimension("R must be a symmetric matrix.")
+            raise ValueError("R must be a symmetric matrix.")
 
         # Create back-up of arrays needed for later computations
         R_ba = copy(R)
@@ -688,10 +688,10 @@ def care_slycot(A, B, Q, R=None, S=None, E=None, stabilizing=True):
             raise ControlDimension("Incompatible dimensions of S matrix.")
 
         if not _is_symmetric(Q):
-            raise ControlDimension("Q must be a symmetric matrix.")
+            raise ValueError("Q must be a symmetric matrix.")
 
         if not _is_symmetric(R):
-            raise ControlDimension("R must be a symmetric matrix.")
+            raise ValueError("R must be a symmetric matrix.")
 
         # Create back-up of arrays needed for later computations
         R_b = copy(R)
@@ -897,10 +897,10 @@ def dare_slycot(A, B, Q, R, S=None, E=None, stabilizing=True):
             raise ControlDimension("Incompatible dimensions of B matrix.")
 
         if not _is_symmetric(Q):
-            raise ControlDimension("Q must be a symmetric matrix.")
+            raise ValueError("Q must be a symmetric matrix.")
 
         if not _is_symmetric(R):
-            raise ControlDimension("R must be a symmetric matrix.")
+            raise ValueError("R must be a symmetric matrix.")
 
         # Create back-up of arrays needed for later computations
         A_ba = copy(A)
@@ -1004,10 +1004,10 @@ def dare_slycot(A, B, Q, R, S=None, E=None, stabilizing=True):
             raise ControlDimension("Incompatible dimensions of S matrix.")
 
         if not _is_symmetric(Q):
-            raise ControlDimension("Q must be a symmetric matrix.")
+            raise ValueError("Q must be a symmetric matrix.")
 
         if not _is_symmetric(R):
-            raise ControlDimension("R must be a symmetric matrix.")
+            raise ValueError("R must be a symmetric matrix.")
 
         # Create back-up of arrays needed for later computations
         A_b = copy(A)
