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

Change considered non impedance branches values in ACOPF #91

Merged
merged 40 commits into from
Jan 8, 2025

Conversation

p-arvy
Copy link
Member

@p-arvy p-arvy commented Dec 19, 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?

The transformer ratios and phase shiftings are considered even on non impedant branches in ACOPF.
For lines considered as non-impedant and connecting two different nominal voltage levels, the conductances and susceptances are cancelled in order to avoid numerical instabilities.

What is the current behavior?

These elements are ignored.
Transformer ratios and phase shiftings are not considered.
Conductances and susceptances are also cancelled for lines connecting same nominal voltage levels.

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

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:

@p-arvy p-arvy requested a review from klorel December 19, 2024 09:44
@p-arvy p-arvy self-assigned this Dec 19, 2024
@p-arvy p-arvy requested a review from So-Fras December 19, 2024 09:44
@p-arvy p-arvy force-pushed the add-branches-opened-one-side branch from f6a2cf5 to 6d8b414 Compare December 19, 2024 16:44
@p-arvy p-arvy force-pushed the add-rho-alpha-for-znull-branches-in-acopf branch from 2e707c3 to 61240d2 Compare December 19, 2024 16:45
@p-arvy p-arvy changed the title Consider transformer ratio and phase shifting on non impedance branch in ACOPF Consider transformer ratio and phase shifting of non impedance branches in ACOPF Dec 19, 2024
@p-arvy p-arvy force-pushed the add-rho-alpha-for-znull-branches-in-acopf branch from 61240d2 to 9aa3c58 Compare December 19, 2024 17:01
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
@p-arvy p-arvy force-pushed the add-branches-opened-one-side branch from f2f5fdd to 9d951c1 Compare December 30, 2024 11:50
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
Signed-off-by: p-arvy <[email protected]>
@p-arvy p-arvy force-pushed the add-rho-alpha-for-znull-branches-in-acopf branch from 9aa3c58 to 1f9960c Compare December 30, 2024 12:04
@p-arvy p-arvy changed the title Consider transformer ratio and phase shifting of non impedance branches in ACOPF Change considered non impedance branches values in ACOPF Dec 30, 2024
Signed-off-by: p-arvy <[email protected]>
@So-Fras So-Fras changed the base branch from add-branches-opened-one-side to main January 8, 2025 14:40
Copy link

sonarqubecloud bot commented Jan 8, 2025

@So-Fras So-Fras merged commit 00686f2 into main Jan 8, 2025
7 checks passed
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.

3 participants