Skip to content

Add LineCouplings extension#3730

Merged
olperr1 merged 24 commits into
mainfrom
mutual-coupling
Jun 17, 2026
Merged

Add LineCouplings extension#3730
olperr1 merged 24 commits into
mainfrom
mutual-coupling

Conversation

@colineplqt

@colineplqt colineplqt commented Jan 22, 2026

Copy link
Copy Markdown
Member

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?

The extension does not exist

What is the new behavior (if this is a feature change)?
A new extension of Network is created to store the mutual couplings existing on the network between two lines that are close.
This can be used for short-circuit calculations.

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

  • Yes
  • No

Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
@sonarqubecloud

Copy link
Copy Markdown

@colineplqt colineplqt marked this pull request as ready for review March 9, 2026 11:29
@jeandemanged

Copy link
Copy Markdown
Member

As it is now the link to the other line is missing. This also raises the question, if the extension is on a Line: which line should hold the mutual coupling ? Maybe this should be a network extension holding a collection of lines mutual couplings instead.

See also https://www.powerworld.com/WebHelp/Content/MainDocumentation_HTML/Mutual_Impedance_Records.htm and also CGMES ShortCircuit profile:

image

Note that these formats also account that portions of the lines may have different mutual couplings, e.g. two lines independent over 20 km, then one same tower over 15 km, then independent again.

@colineplqt

Copy link
Copy Markdown
Member Author

As it is now the link to the other line is missing. This also raises the question, if the extension is on a Line: which line should hold the mutual coupling ? Maybe this should be a network extension holding a collection of lines mutual couplings instead.

See also https://www.powerworld.com/WebHelp/Content/MainDocumentation_HTML/Mutual_Impedance_Records.htm and also CGMES ShortCircuit profile:
image

Note that these formats also account that portions of the lines may have different mutual couplings, e.g. two lines independent over 20 km, then one same tower over 15 km, then independent again.

You are absolutely right. I am going to make the change to make this extension at network level. I will also add the portions of line. It is not in my use case but I thought of adding for each line two percentages indicating the beginning and the end of the mutual coupling. Thank you for checking.

@colineplqt colineplqt force-pushed the mutual-coupling branch 3 times, most recently from f363f26 to 2b7631c Compare April 17, 2026 15:38
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
…extension

Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
@colineplqt colineplqt changed the title Add LineMutualCoupling extension Add LineCouplings extension Apr 20, 2026
@colineplqt

Copy link
Copy Markdown
Member Author

As it is now the link to the other line is missing. This also raises the question, if the extension is on a Line: which line should hold the mutual coupling ? Maybe this should be a network extension holding a collection of lines mutual couplings instead.
See also https://www.powerworld.com/WebHelp/Content/MainDocumentation_HTML/Mutual_Impedance_Records.htm and also CGMES ShortCircuit profile:
image
Note that these formats also account that portions of the lines may have different mutual couplings, e.g. two lines independent over 20 km, then one same tower over 15 km, then independent again.

You are absolutely right. I am going to make the change to make this extension at network level. I will also add the portions of line. It is not in my use case but I thought of adding for each line two percentages indicating the beginning and the end of the mutual coupling. Thank you for checking.

I have one "problem" about having the extension at network level: what should be done if one of the line is removed from the network? I would say that it would be good to remove the associated mutual couplings, but I don't know if it is the best.

Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
@sonarqubecloud

Copy link
Copy Markdown

@colineplqt colineplqt requested review from olperr1 and rolnico April 21, 2026 14:43
@colineplqt colineplqt self-assigned this Jun 3, 2026
…idation in `LineSegment` constraints

Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
@github-project-automation github-project-automation Bot moved this from In Progress to Waiting for review in Release 06/2026 Jun 16, 2026
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Signed-off-by: Coline Piloquet <coline.piloquet@rte-france.com>
Co-authored-by: Olivier Perrin <olivier.perrin@rte-france.com>
Signed-off-by: Coline P <coline.piloquet@rte-france.com>
@sonarqubecloud

Copy link
Copy Markdown

@github-project-automation github-project-automation Bot moved this from Waiting for review to Approved in Release 06/2026 Jun 17, 2026
@olperr1 olperr1 merged commit f15b3d5 into main Jun 17, 2026
9 checks passed
@olperr1 olperr1 deleted the mutual-coupling branch June 17, 2026 14:50
@github-project-automation github-project-automation Bot moved this from Approved to Done in Release 06/2026 Jun 17, 2026
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