Skip to content

Commit

Permalink
remove superfluous use of branch_flows output parameter in tests, closes
Browse files Browse the repository at this point in the history
  • Loading branch information
ccoffrin authored Aug 28, 2023
1 parent a7c6017 commit a8d1aad
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion test/data.jl
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ end


@testset "5-bus case solution with duals" begin
result = run_dc_opf("../test/data/matpower/case5.m", nlp_solver, setting = Dict("output" => Dict("branch_flows" => true, "duals" => true)))
result = run_dc_opf("../test/data/matpower/case5.m", nlp_solver, setting = Dict("output" => Dict("duals" => true)))
result_base = deepcopy(result)

PowerModels.make_mixed_units!(result["solution"])
Expand Down
2 changes: 1 addition & 1 deletion test/pf.jl
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ end
#=
# numerical issues with ipopt, likely div. by zero issue in jacobian
@testset "5-bus case with hvdc line" begin
result = run_pf("../test/data/matpower/case5_dc.m", ACRPowerModel, nlp_solver, setting = Dict("output" => Dict("branch_flows" => true)))
result = run_pf("../test/data/matpower/case5_dc.m", ACRPowerModel, nlp_solver)
@test result["termination_status"] == LOCALLY_SOLVED
@test isapprox(result["objective"], 0; atol = 1e-2)
Expand Down
8 changes: 4 additions & 4 deletions test/warmstart.jl
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ end
@testset "dc warm starts" begin
@testset "5 bus case" begin
data = PowerModels.parse_file("../test/data/matpower/case5.m")
result = run_dc_opf(data, nlp_solver; setting = Dict("output" => Dict("branch_flows" => true)));
result = run_dc_opf(data, nlp_solver)

PowerModels.update_data!(data, result["solution"])

Expand All @@ -55,7 +55,7 @@ end

@testset "5 bus pwl case" begin
data = PowerModels.parse_file("../test/data/matpower/case5_pwlc.m")
result = run_dc_opf(data, nlp_solver; setting = Dict("output" => Dict("branch_flows" => true)));
result = run_dc_opf(data, nlp_solver)

PowerModels.update_data!(data, result["solution"])

Expand All @@ -73,7 +73,7 @@ end
@testset "ac warm starts" begin
@testset "5 bus case" begin
data = PowerModels.parse_file("../test/data/matpower/case5.m")
result = run_ac_opf(data, nlp_solver; setting = Dict("output" => Dict("branch_flows" => true)));
result = run_ac_opf(data, nlp_solver)

PowerModels.update_data!(data, result["solution"])

Expand All @@ -88,7 +88,7 @@ end

@testset "5 bus pwl case" begin
data = PowerModels.parse_file("../test/data/matpower/case5_pwlc.m")
result = run_ac_opf(data, nlp_solver; setting = Dict("output" => Dict("branch_flows" => true)));
result = run_ac_opf(data, nlp_solver)

PowerModels.update_data!(data, result["solution"])

Expand Down

0 comments on commit a8d1aad

Please sign in to comment.