Support three windings transformer in terminals connection action#1338
Support three windings transformer in terminals connection action#1338
Conversation
src/main/java/com/powsybl/openloadflow/network/action/LfTerminalsConnectionAction.java
Outdated
Show resolved
Hide resolved
src/main/java/com/powsybl/openloadflow/network/action/LfTerminalsConnectionAction.java
Outdated
Show resolved
Hide resolved
src/main/java/com/powsybl/openloadflow/network/action/LfTerminalsConnectionAction.java
Outdated
Show resolved
Hide resolved
3da2dec to
74d7459
Compare
Signed-off-by: Bertrand Rix <bertrand.rix@artelys.com>
Signed-off-by: Bertrand Rix <bertrand.rix@artelys.com>
Signed-off-by: Bertrand Rix <bertrand.rix@artelys.com>
This reverts commit dacca70. Signed-off-by: Bertrand Rix <bertrand.rix@artelys.com>
Signed-off-by: Bertrand Rix <bertrand.rix@artelys.com>
50334e7 to
a06c465
Compare
|
| SecurityAnalysisResult result = runSecurityAnalysis(network, contingencies, Collections.emptyList(), securityAnalysisParameters, | ||
| operatorStrategies, actions, ReportNode.NO_OP); | ||
|
|
||
| assertTrue(result.getOperatorStrategyResults().stream().findAny().isPresent()); |
There was a problem hiding this comment.
add assertions on LINE_12 flows via state monitors, here we can see it does not work as expected, because the 3WT opening should reduce LINE_12 flows to zero in this case, and we get 16 MW.
in the logs we see only leg3 is opened - all three legs should be opened:
INFO 18:29:33.146 c.p.o.s.AbstractSecurityAnalysis | Start operator strategy Strategy3WT after contingency 'LOAD_4' simulation on network {CC0 SC0}
TRACE 18:29:33.148 c.p.o.n.LfNetworkListenerTracer | onDisableChange(elementType=BUS, elementId='VL_4_0', disabled=true)
TRACE 18:29:33.150 c.p.o.n.LfNetworkListenerTracer | onDisableChange(elementType=BRANCH, elementId='T3wT_leg_3', disabled=true)
This is because LfTerminalsConnectionAction extends AbstractLfBranchAction, and AbstractLfBranchAction is currently able to handle only one branch, not multiple branches
There was a problem hiding this comment.
also improve this doc section to mention that only branches and three-windings transformer are supported for TerminalsConnectionAction: https://powsybl.readthedocs.io/projects/powsybl-open-loadflow/en/latest/security/inputs.html#implemented-remedial-actions



Please check if the PR fulfills these requirements
Does this PR already have an issue describing the problem?
No issue
What kind of change does this PR introduce?
Support three windings transformers in terminals connection action
What is the current behavior?
Three windings transformers in terminals connection action raise an exception
What is the new behavior (if this is a feature change)?
Three windings transformers are now supported and the three related leg branches can be opened or closed using the action
Does this PR introduce a breaking change or deprecate an API?
If yes, please check if the following requirements are fulfilled
What changes might users need to make in their application due to this PR? (migration steps)
Other information: