Skip to content

Fix the application of validate to powsybl power flow results#56

Open
Tex1718 wants to merge 6 commits into
gridfm:mainfrom
yemine0x01:add-powsybl-powerflow
Open

Fix the application of validate to powsybl power flow results#56
Tex1718 wants to merge 6 commits into
gridfm:mainfrom
yemine0x01:add-powsybl-powerflow

Conversation

@Tex1718

@Tex1718 Tex1718 commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

3 issues:

  • makeYbus needed the bus indexes to be sorted to function correctly
  • update_powsybl did not update the loads correctly
  • power balance cannot be validated, with a persistent small mismatch

Proposed fix:

  • makeYbus is now bus order agnostic
  • update_powsybl updates loads correctly
  • increase atol from 1e-3 to 5e-2 for the validate_power_balance_equations function

@frmir frmir self-assigned this Jun 5, 2026
@frmir frmir requested a review from ttolhurst June 5, 2026 14:38
Tex1718 added 3 commits June 11, 2026 15:39
Signed-off-by: tren <tengxiang.ren@artelys.com>
Signed-off-by: tren <tengxiang.ren@artelys.com>
Signed-off-by: tren <tengxiang.ren@artelys.com>
@Tex1718 Tex1718 force-pushed the add-powsybl-powerflow branch from d6f40f5 to a30b3e6 Compare June 11, 2026 19:39
@Tex1718 Tex1718 force-pushed the add-powsybl-powerflow branch from 461168a to f86c7ed Compare June 17, 2026 15:15
@Tex1718

Tex1718 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

Apologies, accidentally pushed another fix here. Will start a proper PR.

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.

2 participants