Add DC alias for constraint_ohms_y_oltc_pst #880
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Dear @ccoffrin,
Thank you for integrating the constraint_ohms_y_pst constraints into the master. I added alias functions for the DCP formulations of
constraint_ohms_y_oltc_pst_to
andconstraint_ohms_y_oltc_pst_from
functions which refer to the newly addedconstraint_ohms_y_pst_to
andconstraint_ohms_y_pst_from
in dcp.jl.In this way, it suffices to call constraint_ohms_y_oltc_pst from constraint_template.jl for both AC and DC formulations. The DC formulation obviously lacks the possibility to change the voltages by adjusting the OLTC (tm). But this should be clear if the DC models are used.
Best regards,
Martin