Fix verbose option

audiolabs · Oct 4, 2024 · 6754ead · 6754ead
1 parent dfd3fcb
commit 6754ead
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/anf_generator/MixingMatrix.py b/anf_generator/MixingMatrix.py
@@ -357,7 +357,7 @@ def absoluteUnitaryProcrustes(A, B):
 
         for _ in range(MaximumTrails):
             # Perform optimization
-            U = MixingMatrix.signVariableExchange(A, newPhases, True)
+            U = MixingMatrix.signVariableExchange(A, newPhases, False)
 
             # Evaluate l1-norm of obtained unitary matrix
             l1 = MixingMatrix.l1norm(U @ A)
@@ -387,7 +387,7 @@ def absoluteOrthogonalProcrustes(A, B):
 
         for _ in range(MaximumTrails):
             # Perform optimization
-            U = MixingMatrix.signVariableExchange(A, newSigns, True)
+            U = MixingMatrix.signVariableExchange(A, newSigns, False)
 
             # Evaluate l1-norm of obtained unitary matrix
             l1 = MixingMatrix.l1norm(U @ A)
@@ -446,10 +446,10 @@ def signVariableExchange(A, B, verbose=False):
 
             # Check convergence
             if delta > np.sqrt(np.finfo(float).eps):
-                if not verbose:
+                if verbose:
                     print("Variable exchange optimization did not improve.")
             elif delta > -1e-5:
-                if not verbose:
+                if verbose:
                     print(
                         f"Variable exchange optimization converged within {counter} iterations."
                     )