formatting and comment for posterity

Author: marcomangano

tests/test_hs071.py

@@ -222,10 +222,14 @@ def test_optimization(self, optName):
         lambda_con1 = float(lines[con1_line_num].split()[-1])
         lambda_con2 = float(lines[con2_line_num].split()[-1])
         if optName in ("IPOPT", "SNOPT", "ParOpt"):
-            lambda_sign = -1.0 if optName == 'IPOPT' else 1.0
-            assert_allclose([lambda_con1, lambda_con2],
-                            lambda_sign * np.asarray(self.lambdaStar[0]["con"]),
-                            rtol=1.0e-5, atol=1.0e-5)
+            # IPOPT returns Lagrange multipliers with opposite sign than SNOPT and ParOpt
+            lambda_sign = -1.0 if optName == "IPOPT" else 1.0
+            assert_allclose(
+                [lambda_con1, lambda_con2],
+                lambda_sign * np.asarray(self.lambdaStar[0]["con"]),
+                rtol=1.0e-5,
+                atol=1.0e-5,
+            )
         else:
             assert_allclose([lambda_con1, lambda_con2], [9.0e100, 9.0e100], rtol=1.0e-5, atol=1.0e-5)
 

tests/testing_utils.py

@@ -144,9 +144,7 @@ def assert_solution_allclose(self, sol, tol, partial_x=False, lambda_sign=1.0):
             and sol.lambdaStar is not None
         ):
             lamStar = {con: lambda_sign * lam for con, lam in sol.lambdaStar.items()}
-            assert_dict_allclose(lamStar,
-                                 self.lambdaStar[self.sol_index],
-                                 atol=tol, rtol=tol)
+            assert_dict_allclose(lamStar, self.lambdaStar[self.sol_index], atol=tol, rtol=tol)
 
         # test printing solution
         print(sol)