Skip to content

Support having multiple active operational limits group#1345

Merged
SylvestreSakti merged 25 commits intomainfrom
multiple_limits_group
Mar 24, 2026
Merged

Support having multiple active operational limits group#1345
SylvestreSakti merged 25 commits intomainfrom
multiple_limits_group

Conversation

@SylvestreSakti
Copy link
Copy Markdown
Member

@SylvestreSakti SylvestreSakti commented Feb 5, 2026

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • A PR or issue has been opened in all impacted repositories (if any)

Does this PR already have an issue describing the problem?
Closes #1356

What kind of change does this PR introduce?
powsybl-core PR powsybl/powsybl-core#3735 will introduce multiple operational limits groups. This current PR aims at supporting this new functionnality.

What is the current behavior?
Only one operational limits group can be selected

What is the new behavior (if this is a feature change)?
Multiple operational limits group can be selected. A new class LfLimitsGroup (which only contains a list of LfLimit objects) has been introduced to avoid using nested lists.
The operationalLimitsGroupId is added to the LimitViolations that are in the result

Does this PR introduce a breaking change or deprecate an API?

  • Yes
  • No

If yes, please check if the following requirements are fulfilled

  • The Breaking Change or Deprecated label has been added
  • The migration steps are described in the following section

What changes might users need to make in their application due to this PR? (migration steps)
For custom open-loadflow implementation using LfNetwork, the LfBranch API has been modified :
getLimits1() and getLimits2() return now a List<LfLimitsGroup> instead of a List<LfLimit>

Other information:

Signed-off-by: PRABAKARAN Sylvestre <sylvestre.prabakaran@rte-france.com>
Signed-off-by: PRABAKARAN Sylvestre <sylvestre.prabakaran@rte-france.com>
Signed-off-by: PRABAKARAN Sylvestre <sylvestre.prabakaran@rte-france.com>
Signed-off-by: PRABAKARAN Sylvestre <sylvestre.prabakaran@rte-france.com>
Signed-off-by: PRABAKARAN Sylvestre <sylvestre.prabakaran@rte-france.com>
Copy link
Copy Markdown

@NathanDissoubray NathanDissoubray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me appart from a little question on the order of the limits 👍

Have you taken a look at powsybl/powsybl-core#3756 ? I don't think this impacts OLF, but I prefer making sure

Comment thread src/main/java/com/powsybl/openloadflow/network/LfBranch.java
Comment thread src/main/java/com/powsybl/openloadflow/network/LfBranch.java
SylvestreSakti and others added 10 commits March 9, 2026 10:33
Signed-off-by: PRABAKARAN Sylvestre <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
…(powsybl-core 3670) (#1375)

Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
Signed-off-by: Damien Jeandemange <damien.jeandemange@artelys.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
@SylvestreSakti SylvestreSakti marked this pull request as ready for review March 20, 2026 19:09
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Copy link
Copy Markdown

@NathanDissoubray NathanDissoubray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me. Some of the things you had to implement should've been made on core instead, we'll see what we can do regarding that !

@sonarqubecloud
Copy link
Copy Markdown

@SylvestreSakti SylvestreSakti merged commit 879e8f7 into main Mar 24, 2026
7 checks passed
@SylvestreSakti SylvestreSakti deleted the multiple_limits_group branch March 24, 2026 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support having multiple operational limits groups active at the same time

4 participants