Skip to content
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

DC Area Interchange Control OuterLoop #1123

Merged
merged 17 commits into from
Dec 3, 2024
Merged

Conversation

vmouradian
Copy link
Member

@vmouradian vmouradian commented Nov 18, 2024

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)

Does this PR already have an issue describing the problem?

No

What kind of change does this PR introduce?

Feature

What is the current behavior?

Area interchange control is not supported for DC load flow

What is the new behavior (if this is a feature change)?
Area interchange is supported, via a new DC outerloop, that shares most of the logic with the AC version.

Introduces :

  • refactor, classes/interfaces to share code between DistributedSlack and AC/DC AIC outerloops
  • Result for distributed active power in DC LF

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)

Other information:

AC AIC OuterLoop was done in #1055

@vmouradian vmouradian self-assigned this Nov 18, 2024
@vmouradian vmouradian changed the base branch from feature/dc_multiple_outerloops to main November 18, 2024 14:04
@vmouradian vmouradian changed the base branch from main to feature/dc_multiple_outerloops November 18, 2024 14:15
@vidaldid-rte
Copy link
Collaborator

Can you add something in the doc that says that Area Interchange control works in DC and AC ?
And with the limitations if any.

@vmouradian vmouradian force-pushed the feature/dc-aic-outer-loop branch from 4830f6d to 42658e2 Compare November 19, 2024 17:49
Base automatically changed from feature/dc_multiple_outerloops to main November 20, 2024 10:41
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
@vmouradian vmouradian force-pushed the feature/dc-aic-outer-loop branch from ff33b66 to 2353626 Compare November 21, 2024 15:48
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
# Conflicts:
#	src/main/java/com/powsybl/openloadflow/OpenLoadFlowParameters.java
Signed-off-by: Damien Jeandemange <damien.jeandemange@artelys.com>
Signed-off-by: Damien Jeandemange <damien.jeandemange@artelys.com>
@jeandemanged jeandemanged merged commit 6fab040 into main Dec 3, 2024
8 checks passed
@jeandemanged jeandemanged deleted the feature/dc-aic-outer-loop branch December 3, 2024 15:10
@jeandemanged jeandemanged changed the title DC area interchange control outerloop DC Area Interchange Control OuterLoop Dec 3, 2024
vidaldid-rte pushed a commit that referenced this pull request Dec 11, 2024
Signed-off-by: vmouradian <valentin.mouradian@artelys.com>
Signed-off-by: Didier Vidal <didier.vidal_externe@rte-france.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants