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

fix unbalanced powerflow with out of service ext grids #2227

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

jkupka
Copy link
Collaborator

@jkupka jkupka commented Feb 29, 2024

Solves a mismatch of arrays during zero impedance addition of ext grids if at least one ext grid is out of service

Copy link

codecov bot commented Feb 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.66%. Comparing base (f2794c6) to head (4613282).
Report is 34 commits behind head on develop.

Current head 4613282 differs from pull request most recent head c289f52

Please upload reports for the commit c289f52 to get more accurate results.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop    #2227       +/-   ##
============================================
+ Coverage    62.30%   79.66%   +17.36%     
============================================
  Files          255      255               
  Lines        28062    27840      -222     
============================================
+ Hits         17483    22178     +4695     
+ Misses       10579     5662     -4917     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jkupka jkupka requested a review from dlohmeier March 6, 2024 08:28
@rbolgaryn
Copy link
Member

This one was merged: fix unbalanced powerflow with out ouf service ext grids #2184

Is this PR still relevant?

@vogt31337
Copy link
Contributor

@jkupka @dlohmeier still relevant?

@jkupka
Copy link
Collaborator Author

jkupka commented Mar 28, 2024

yes, it's still relevant as it solves a different issue than #2184

Copy link
Member

@rbolgaryn rbolgaryn left a comment

Choose a reason for hiding this comment

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

please address the comment from Daniel and add information to CHANGELOG.
Please also write a test that fails with the current pandapower and passes because with this pull request.

@rbolgaryn
Copy link
Member

@jkupka please address the change request

@jkupka jkupka requested a review from rbolgaryn May 24, 2024 12:56
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.

None yet

4 participants