Skip to content

cleanup opf in case of non convergence - #3084

Open
marakasm wants to merge 2 commits into
e2nIEE:developfrom
marakasm:develop
Open

cleanup opf in case of non convergence#3084
marakasm wants to merge 2 commits into
e2nIEE:developfrom
marakasm:develop

Conversation

@marakasm

@marakasm marakasm commented Aug 4, 2026

Copy link
Copy Markdown

Hello, this is my first PR, so don't hesitate to let me know if I need to add something, for example file an issue linked to this.

Optimal power flow was not cleaned up in case it did not converge. This implied that generators created to model dclines were not deleted. This is a problem if another opf is run afterwards.

@sonarqubecloud

sonarqubecloud Bot commented Aug 4, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.12%. Comparing base (44d7d69) to head (2d4fbaa).

Files with missing lines Patch % Lines
pandapower/optimal_powerflow.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3084      +/-   ##
===========================================
- Coverage    72.12%   72.12%   -0.01%     
===========================================
  Files          355      355              
  Lines        39376    39377       +1     
===========================================
  Hits         28400    28400              
- Misses       10976    10977       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vogt31337

Copy link
Copy Markdown
Contributor

@marakasm , thanks for the PR. Does IT solve #3018 ?
I'll have a Look into IT later.

@marakasm

marakasm commented Aug 5, 2026

Copy link
Copy Markdown
Author

@marakasm , thanks for the PR. Does IT solve #3018 ? I'll have a Look into IT later.

@vogt31337 I dont think this is related, I did not work on testing, and did not change convergence behavior, just added a clean up in case of non convergence.

Thanks !

@KS-HTK

KS-HTK commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

Does this solve the issue described in #3073 ?

@marakasm

marakasm commented Aug 6, 2026

Copy link
Copy Markdown
Author

Does this solve the issue described in #3073 ?

I don't think so because the clean up that I added is related to hvdc lines and the example used in #3073 does not contain any if I am not mistaken

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