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

CGMES export SV: voltage of tie line boundary node #2841

Merged
merged 2 commits into from
Jan 3, 2024

Conversation

zamarrenolm
Copy link
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)

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

Topological node for boundary point of tie line was read from IIDM TieLine alias. Now that tie line is only a container of underlying tie lines, this data should be read from the inner dangling lines.

What is the new behavior (if this is a feature change)?
Topological node for boundary point of tie line is read from IIDM property of dangling lines of TieLine.

Sorry, something went wrong.

Verified

This commit was signed with the committer’s verified signature.
Kobzol Jakub Beránek
Signed-off-by: Luma <zamarrenolm@aia.es>

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Copy link

sonarqubecloud bot commented Jan 3, 2024

@annetill annetill merged commit ed7694e into main Jan 3, 2024
6 checks passed
@annetill annetill deleted the cgmes_export_sv_tie_line_boundary_voltage branch January 3, 2024 11:00
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.

None yet

2 participants