add test and fix to calc_theta_delta_bounds for multiconductor case (#…

…871)
lanl-ansi · Aug 11, 2023 · eb21c13 · eb21c13
1 parent e9712cd
commit eb21c13
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,7 +2,7 @@ PowerModels.jl Change Log
 =========================
 
 ### Staged
-- nothing
+- Fix implementation of `calc_theta_delta_bounds` when conductor parameter is used (#870)
 
 ### v0.19.9
 - Fix implementation of polynomial costs above quadratic

diff --git a/src/core/data.jl b/src/core/data.jl
@@ -73,7 +73,7 @@ function calc_theta_delta_bounds(data::Dict{String,<:Any})
     end
 
     if haskey(data, "conductors")
-        return amin, amax
+        return angle_min, angle_max
     else
         return angle_min[1], angle_max[1]
     end

diff --git a/test/data-mc.jl b/test/data-mc.jl
@@ -0,0 +1,13 @@
+@testset "test multiconductor data" begin
+
+    @testset "multiconductor calc_theta_delta_bounds" begin
+        data = PowerModels.parse_file("../test/data/matpower/case5.m")
+        data["conductors"] = 1
+
+        theta_lb, theta_ub = calc_theta_delta_bounds(data)
+
+        @test theta_lb[1] <= -2.0
+        @test theta_ub[1] >=  2.0
+    end
+
+end
diff --git a/test/runtests.jl b/test/runtests.jl
@@ -54,6 +54,8 @@ include("common.jl")
 
     include("data.jl")
 
+    include("data-mc.jl")
+
     include("data-basic.jl")
 
     include("model.jl")