Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem with initialization of voltage variables #358

Closed
keatsig opened this issue Aug 31, 2021 · 0 comments
Closed

Problem with initialization of voltage variables #358

keatsig opened this issue Aug 31, 2021 · 0 comments
Labels
Type: Bug Something isn't working

Comments

@keatsig
Copy link
Collaborator

keatsig commented Aug 31, 2021

The seems to be a problem with initializing voltage variables using add_voltage_starts!() .

In the variable_mc_bus_voltage() function for ACP/ACR formulations, there is

vm = haskey(busref, "vm_start") ? busref["vm_start"] : fill(0.0, ncnd) 
vm[.!grounded] .= 1.0

While the first line reads the start values defined by add_voltage_starts!(), it looks like the second line overwrites and reassigns vm to 1.0 (for test cases where most of the buses are ungrounded).

Maybe this could work (might not be ideal):

vm = haskey(busref, "vm_start") ? busref["vm_start"] : fill(1.0, ncnd) 
@keatsig keatsig added the Type: Bug Something isn't working label Aug 31, 2021
@keatsig keatsig changed the title [BUG] Initialization of voltage variables Problem with initialization of voltage variables Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant