Skip to content

Add missing migration on calculated buses#198

Merged
antoinebhs merged 2 commits into
mainfrom
add-missing-migration
May 11, 2026
Merged

Add missing migration on calculated buses#198
antoinebhs merged 2 commits into
mainfrom
add-missing-migration

Conversation

@antoinebhs
Copy link
Copy Markdown
Contributor

@antoinebhs antoinebhs commented May 5, 2026

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines

Does this PR already have an issue describing the problem?

No

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

Can't read voltage level because connectable type "DANGLING_LINE" does not exist anymore in the calculated bus view.

What is the new behavior (if this is a feature change)?
Voltage level connectable type "DANGLING_LINE" is migrated to "BOUNDARY_LINE".
Voltage level can be read even if a boundary line is in the network.

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

  • Yes
  • No

Signed-off-by: BOUHOURS Antoine <antoine.bouhours@rte-france.com>
@antoinebhs antoinebhs force-pushed the add-missing-migration branch from cab6fd8 to f426602 Compare May 5, 2026 10:00
@antoinebhs antoinebhs requested a review from etiennehomer May 5, 2026 10:03
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Collaborator

@EtienneLt EtienneLt left a comment

Choose a reason for hiding this comment

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

Code Ok

@jonenst
Copy link
Copy Markdown
Contributor

jonenst commented May 11, 2026

A bit sad to do seds on this data but it works, and the volumes are compatible with a synchronous migration so it should be ok

@antoinebhs antoinebhs merged commit a3b31ba into main May 11, 2026
5 checks passed
@antoinebhs antoinebhs deleted the add-missing-migration branch May 11, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants