Skip to content

Refactor DistributedSlackOuterLoop reports when handling several SC - #1473

Open
bperr wants to merge 1 commit into
mainfrom
fix-report-node-several-synchronous-component
Open

Refactor DistributedSlackOuterLoop reports when handling several SC#1473
bperr wants to merge 1 commit into
mainfrom
fix-report-node-several-synchronous-component

Conversation

@bperr

@bperr bperr commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

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)

What kind of change does this PR introduce?

Minor feature change, not covered in the last PR : clean reports for distributed slack outer loop when there are several SC

What is the current behavior?

Report structure is

+ Outer loop DistributedSlack
   + SC0
      + Outer loop iteration 1
         Slack bus active power (40 MW) distributed in 1 distribution iteration(s)
   + SC1
      + Outer loop iteration 1
         Slack bus active power (-19.998222 MW) distributed in 1 distribution iteration(s)

What is the new behavior (if this is a feature change)?
New structure is

+ Outer loop DistributedSlack
   + Outer loop iteration 1
      + SC0
         Slack bus active power (40 MW) distributed in 1 distribution iteration(s)
      + SC1
         Slack bus active power (-19.998222 MW) distributed in 1 distribution iteration(s)

Please not that a SC is not added if there is no report inside.

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

  • Yes
  • No

…hronous component

Signed-off-by: b.perreyon <baptiste.perreyon@supergrid-institute.com>
@sonarqubecloud

Copy link
Copy Markdown

@bperr
bperr marked this pull request as ready for review July 27, 2026 15:16

@SylvestreSakti SylvestreSakti left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for solving this !

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.

2 participants