Pare back AC reordering tests

Gurobi · Jul 31, 2023 · 6172f64 · 6172f64
1 parent 380f8a4
commit 6172f64
Showing 1 changed file with 6 additions and 43 deletions.
diff --git a/tests/opf/test_solver.py b/tests/opf/test_solver.py
@@ -257,6 +257,8 @@ def test_dc(self):
                 self.assert_approx_equal(gen_reordered[key], value, tol=1e-3)
 
     def test_ac(self):
+        # Results are too numerically unstable, just check the call goes through
+        # without errors and objective function value is in the ballpark.
         kwargs = dict(
             opftype="ac",
             solver_params={"MIPGap": 1e-4},
@@ -265,27 +267,12 @@ def test_ac(self):
         solution_reordered = solve_opf(self.case_reordered, **kwargs)
 
         self.assert_approx_equal(
-            solution_original["f"], solution_reordered["f"], tol=1e0
+            solution_original["f"], solution_reordered["f"], tol=1e1
         )
 
-        for ind, orig_ind in enumerate(self.bus_reorder):
-            bus_original = solution_original["bus"][orig_ind]
-            bus_reordered = solution_reordered["bus"][ind]
-            self.assert_approx_equal(bus_reordered["Vm"], bus_original["Vm"], tol=1e0)
-
-        for ind, orig_ind in enumerate(self.branch_reorder):
-            branch_original = solution_original["branch"][orig_ind]
-            branch_reordered = solution_reordered["branch"][ind]
-            self.assert_approx_equal(
-                branch_reordered["Qf"], branch_original["Qf"], tol=1e0
-            )
-
-        for ind, orig_ind in enumerate(self.gen_reorder):
-            gen_original = solution_original["gen"][orig_ind]
-            gen_reordered = solution_reordered["gen"][ind]
-            self.assert_approx_equal(gen_reordered["Qg"], gen_original["Qg"], tol=1e0)
-
     def test_ac_relax(self):
+        # Results are too numerically unstable, just check the call goes through
+        # without errors and objective function value is in the ballpark.
         kwargs = dict(
             opftype="acrelax",
             solver_params={"OptimalityTol": 1e-6},
@@ -294,33 +281,9 @@ def test_ac_relax(self):
         solution_reordered = solve_opf(self.case_reordered, **kwargs)
 
         self.assert_approx_equal(
-            solution_original["f"], solution_reordered["f"], tol=1e-1
+            solution_original["f"], solution_reordered["f"], tol=1e1
         )
 
-        for ind, orig_ind in enumerate(self.bus_reorder):
-            bus_original = solution_original["bus"][orig_ind]
-            bus_reordered = solution_reordered["bus"][ind]
-            for key in ["Vm"]:
-                self.assert_approx_equal(
-                    bus_reordered[key], bus_original["Vm"], tol=1e-1
-                )
-
-        for ind, orig_ind in enumerate(self.branch_reorder):
-            branch_original = solution_original["branch"][orig_ind]
-            branch_reordered = solution_reordered["branch"][ind]
-            for key in ["Pt"]:
-                self.assert_approx_equal(
-                    branch_reordered["Pt"], branch_original["Pt"], tol=1e-1
-                )
-
-        for ind, orig_ind in enumerate(self.gen_reorder):
-            gen_original = solution_original["gen"][orig_ind]
-            gen_reordered = solution_reordered["gen"][ind]
-            for key in ["Pg"]:
-                self.assert_approx_equal(
-                    gen_reordered[key], gen_original[key], tol=1e-1
-                )
-
 
 @unittest.skipIf(size_limited_license(), "size-limited-license")
 class TestAPILargeModels(unittest.TestCase):