powsybl / powsybl-core

A framework to build power system oriented software
https://www.powsybl.org
Mozilla Public License 2.0
124 stars 40 forks source link

Consider sides when applying limitReductions with IdentifiableCriterion #3050

Closed olperr1 closed 3 months ago

olperr1 commented 4 months ago

Please check if the PR fulfills these requirements

Does this PR already have an issue describing the problem?

No

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

When a LimitReduction is defined using an IdentifiableCriterion (to define a reduction on a voltage level for instance), the reduction is applied on each side of the network element if one of its sides verify the criterion.

What is the new behavior (if this is a feature change)? For IdentifiableCriterion, the reduction is applied only on the side(s) verifying the criterion.

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:

sonarcloud[bot] commented 3 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
88.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud