-
Notifications
You must be signed in to change notification settings - Fork 43
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
ADD: Capacitor control for IVR and FOTP formulations #404
Conversation
Codecov Report
@@ Coverage Diff @@
## main #404 +/- ##
==========================================
+ Coverage 69.69% 70.14% +0.44%
==========================================
Files 59 59
Lines 14652 14831 +179
==========================================
+ Hits 10212 10403 +191
+ Misses 4440 4428 -12
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
src/form/ivr.jl
Outdated
""" | ||
variable_mc_capcontrol(pm::AbstractUnbalancedIVRModel; nw::Int=nw_id_default, relax::Bool=false, report::Bool=true) | ||
|
||
Capacitor switching variables. | ||
""" | ||
function variable_mc_capcontrol(pm::AbstractUnbalancedIVRModel; nw::Int=nw_id_default, relax::Bool=false, report::Bool=true) | ||
variable_mc_capacitor_switch_state(pm; nw=nw, relax=relax, report=report) | ||
end | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this necessary? Is having a function with pm::AbstractUnbalancedPowerModels
would be enough, I don't see any custom version of variable_mc_capacitor_switch_state
for IVR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might not be necessary. I think the other formulations (ACP, ACR) also have individual variable_mc_capcontrol
but call the same variable_mc_capacitor_switch_state
function. I can modify this and create a single variable_mc_capcontrol(pm::AbstractUnbalancedPowerModels;...)
that should work for all formulations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, let's do that. the current way seems redundant, and this would reduce the amount of code we need to maintain in the future
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes have been made. Also, FOTP formulation did not have CapControl and the feature has now been added (changed the PR description accordingly).
This PR adds capacitor control to current-voltage variable space (IVR) and first-order Taylor polar (FOTP) formulations. This control emulates a typical utility wye-connected capacitor by sending switching messages.