Skip to content

Documentation: Improve Power Flow Method Comparisions#1306

Merged
nitbharambe merged 14 commits intomainfrom
documentation/improve_pf_documentation_01
Mar 31, 2026
Merged

Documentation: Improve Power Flow Method Comparisions#1306
nitbharambe merged 14 commits intomainfrom
documentation/improve_pf_documentation_01

Conversation

@Jerry-Jinfeng-Guo
Copy link
Copy Markdown
Member

Improvements as requested in #1003

  • Methods comparision

Signed-off-by: Jerry Jinfeng Guo <jerry.jinfeng.guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo self-assigned this Feb 19, 2026
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo added the documentation Improvements or additions to documentation label Feb 19, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the user manual’s power flow documentation to help users choose an appropriate calculation method (per #1003) by adding a more practical comparison and selection guidance.

Changes:

  • Replaces the power flow algorithm comparison table with columns focused on convergence and typical use cases.
  • Adds a “Choosing the right power flow algorithm” section with selection guidance based on accuracy, speed, and grid characteristics.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo marked this pull request as draft February 20, 2026 08:49
Copy link
Copy Markdown
Member

@figueroa1395 figueroa1395 left a comment

Choose a reason for hiding this comment

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

Looks good to me. Nothing more to add besides what was already pointed out.

Jerry-Jinfeng-Guo and others added 5 commits March 2, 2026 13:50
Co-authored-by: Nitish Bharambe <78108900+nitbharambe@users.noreply.github.com>
Signed-off-by: Jerry Guo <6221579+Jerry-Jinfeng-Guo@users.noreply.github.com>
Co-authored-by: Nitish Bharambe <78108900+nitbharambe@users.noreply.github.com>
Signed-off-by: Jerry Guo <6221579+Jerry-Jinfeng-Guo@users.noreply.github.com>
Co-authored-by: Nitish Bharambe <78108900+nitbharambe@users.noreply.github.com>
Signed-off-by: Jerry Guo <6221579+Jerry-Jinfeng-Guo@users.noreply.github.com>
Signed-off-by: Jerry Jinfeng Guo <jerry.jinfeng.guo@alliander.com>
Signed-off-by: Jerry Jinfeng Guo <jerry.jinfeng.guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo marked this pull request as ready for review March 3, 2026 15:42
Signed-off-by: Nitish Bharambe <nitish.bharambe@alliander.com>
Signed-off-by: Nitish Bharambe <nitish.bharambe@alliander.com>
@nitbharambe nitbharambe disabled auto-merge March 27, 2026 12:17
Copy link
Copy Markdown
Member

@figueroa1395 figueroa1395 left a comment

Choose a reason for hiding this comment

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

Minor comments only. Looks good to me.

@figueroa1395
Copy link
Copy Markdown
Member

@nitbharambe should #1316 follow after this one? Do you think it isn't needed? Or what's the status there?

Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
nitbharambe and others added 2 commits March 30, 2026 17:02
Co-authored-by: Santiago Figueroa Manrique <figueroa1395@gmail.com>
Co-authored-by: Martijn Govers <martygovers@hotmail.com>
Signed-off-by: Nitish Bharambe <78108900+nitbharambe@users.noreply.github.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
mgovers
mgovers previously approved these changes Mar 31, 2026
Copy link
Copy Markdown
Member

@mgovers mgovers left a comment

Choose a reason for hiding this comment

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

OK with the current state but i have 2 minor comments. feel free to resolve them if you disagree

figueroa1395
figueroa1395 previously approved these changes Mar 31, 2026
Copy link
Copy Markdown
Member

@figueroa1395 figueroa1395 left a comment

Choose a reason for hiding this comment

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

LGTM. No more comments from my side.

Signed-off-by: Nitish Bharambe <nitish.bharambe@alliander.com>
@nitbharambe nitbharambe dismissed stale reviews from figueroa1395 and mgovers via e1fb0a5 March 31, 2026 08:08
Signed-off-by: Nitish Bharambe <78108900+nitbharambe@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

@nitbharambe nitbharambe enabled auto-merge March 31, 2026 09:30
@nitbharambe nitbharambe added this pull request to the merge queue Mar 31, 2026
Merged via the queue into main with commit cf4ffc3 Mar 31, 2026
31 checks passed
@nitbharambe nitbharambe deleted the documentation/improve_pf_documentation_01 branch March 31, 2026 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants