Skip to content

Conversation

@Lisrte
Copy link
Contributor

@Lisrte Lisrte commented Dec 11, 2025

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?

No

What kind of change does this PR introduce?

Feature

What is the current behavior?

What is the new behavior (if this is a feature change)?
Handle powsybl-dynawo extensions:

  • GeneratorConnectionLevel
  • SynchronizedGeneratorProperties
  • SynchronousGeneratorProperties

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

  • Yes
  • No

Other information:

⚠️ Remove the dynawo extension dependencies version before merging

@Lisrte Lisrte self-assigned this Dec 11, 2025
@Lisrte Lisrte marked this pull request as ready for review December 12, 2025 07:47
@Lisrte Lisrte requested a review from HugoKulesza December 12, 2025 08:04
@alicecaron alicecaron moved this from TODO to In Progress in Release 12/2025 Dec 31, 2025
@alicecaron alicecaron moved this from TODO to In Progress in Release 03/2026 Dec 31, 2025
Copy link
Collaborator

@HugoKulesza HugoKulesza left a comment

Choose a reason for hiding this comment

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

The code is OK, I think that it could be a good idea to precise in the ExtensionInformation that these are extensions to be used in dynamic simulation (and not used by all simulations)

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 7, 2026

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 7, 2026

@Lisrte Lisrte requested a review from HugoKulesza January 8, 2026 08:45
@Lisrte
Copy link
Contributor Author

Lisrte commented Jan 8, 2026

The code is OK, I think that it could be a good idea to precise in the ExtensionInformation that these are extensions to be used in dynamic simulation (and not used by all simulations)

Done

@github-project-automation github-project-automation bot moved this from In Progress to Approved in Release 03/2026 Jan 8, 2026
@HugoKulesza HugoKulesza merged commit 7fcc5d5 into main Jan 8, 2026
10 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in Release 03/2026 Jan 8, 2026
@HugoKulesza HugoKulesza deleted the tests_dynaswing branch January 8, 2026 09:01
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.

5 participants