Skip to content

[WIP] AC OPF proto with pyoptinterface #1071

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 162 commits into
base: main
Choose a base branch
from
Open

[WIP] AC OPF proto with pyoptinterface #1071

wants to merge 162 commits into from

Conversation

@geofjamg
Copy link
Member

@geofjamg geofjamg commented Sep 3, 2025

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)
  • A PR or issue has been opened in all impacted repositories (if any)

Does this PR already have an issue describing the problem?

What kind of change does this PR introduce?

What is the current behavior?

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change or deprecate an API?

  • Yes
  • No

If yes, please check if the following requirements are fulfilled

  • The Breaking Change or Deprecated label has been added
  • The migration steps are described in the following section

What changes might users need to make in their application due to this PR? (migration steps)

Other information:

geofjamg and others added 30 commits March 16, 2025 20:52
@geofjamg
Wip
131e2b4 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
9c817cc 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
a1fa9c7 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Fix
bbf1a97 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Compare with OLF
c6b4beb 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
6ce1609 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Get tap phase shift from transformer dataframes
938a519 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@HugoKulesza
Merge branch 'main' into transfo_alpha
d74fc7f
@geofjamg
Add same reactive limits to generators, VSCs and batteries
a65257e 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Refactor OPF functionality into structured module
818be2c 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
c0bdb48 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
50c495b 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
0d05b07 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
0cee9a6 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
4560630 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Merge branch 'main' into opf_proto
b3ffa9d
@geofjamg
Merge branch 'transfo_alpha' into opf_proto
39b4dff
@geofjamg
Wip
23aedd2 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
32417b6 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Use losses as a cost function
6bec168 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Support open branches
2b567f4 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Fix
01b9742 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
4bcc4fe 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
3c17a40 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
2c7ec9d 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
ef1cd08 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
fca76e4 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Logs variables bounds
0923684 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Use min_q_at_target_p instead of min_q
0d09d87 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
9459517 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Merge branch 'main' into opf_proto
f6339e1
@geofjamg
Fix unit test
d0868ba 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Migrate to pyopt 0.5.0
5bd3af8 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Copy link
Member Author

geofjamg commented Sep 20, 2025

Hello, I am the author of PyOptInterface.

Thanks for your interests in PyOptInterface (POI) as the modeling tool for OPF.

I notice that you are using the old NLP interface in previous version of POI, and I recommend using the more intuitive interface we designed in POI v0.5.0, where you can declare nonlinear constraint directly without specifying function in advance. You can refer to the NLP interface and the ACOPF example.

If you have any questions for the usage of POI, welcome to open a thread in Discussion of POI repo or ping me here.

Hi,
I migrated to 0.5.0.
Thanks for this amazing project!

geofjamg and others added 22 commits September 21, 2025 22:14
@geofjamg
Migrate to pyopt 0.5.1
721edf4 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Merge branch 'main' into opf_proto
ec56d5f
@geofjamg
Wip
a5b04dc 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Add VoltageRegulation
aaefacb 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Read batteries
cc9dfe5 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
b7d896d 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Battery voltage regulation extension support
78d36ce 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Merge branch 'voltage_regulation_extension' into opf_proto
4027107 
# Conflicts:
#	java/pypowsybl/src/main/java/com/powsybl/dataframe/network/extensions/VoltageRegulationDataframeAdder.java
#	java/pypowsybl/src/main/java/com/powsybl/dataframe/network/extensions/VoltageRegulationDataframeProvider.java
@geofjamg
Wip
f2d623f 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
b9d1a39 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
63195df 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
2575f1a 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
c619e96 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
c104006 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
f013e48 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
254b45b 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Wip
10e93a0 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Merge branch 'main' into opf_proto
e2603d3 
# Conflicts:
#	java/pypowsybl/src/main/java/com/powsybl/dataframe/network/extensions/VoltageRegulationDataframeAdder.java
#	java/pypowsybl/src/main/java/com/powsybl/dataframe/network/extensions/VoltageRegulationDataframeProvider.java
#	tests/test_network_extensions.py
@geofjamg
Wip
f1417df 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
@geofjamg
Merge branch 'main' into opf_proto
a75ddd1 
# Conflicts:
#	java/pypowsybl/src/main/java/com/powsybl/dataframe/network/extensions/VoltageRegulationDataframeAdder.java
#	java/pypowsybl/src/main/java/com/powsybl/dataframe/network/extensions/VoltageRegulationDataframeProvider.java
#	tests/test_network_extensions.py
@geofjamg
Merge remote-tracking branch 'origin/opf_proto' into opf_proto
b5e2979 
# Conflicts:
#	tests/test_network_extensions.py
@geofjamg
Adapt to new operational limits dataframe
6671bcb 
Signed-off-by: Geoffroy Jamgotchian <[email protected]>
2nisb
2nisb reviewed Nov 14, 2025
View reviewed changes
pypowsybl/opf/impl/model/variable_context.py
closed_branch_nums = []
open_side1_branch_nums = []
open_side2_branch_nums = []
branch_count = len(network_cache.lines) + len(network_cache.transformers_2w)
Copy link
Contributor

@2nisb 2nisb Nov 14, 2025

Choose a reason for hiding this comment

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

I don't understand why branch_count is not defined as branch_count = len(network_cache.branches) ?

Copy link
Member Author

@geofjamg geofjamg Nov 19, 2025

Choose a reason for hiding this comment

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

We can, this is equivalent

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 19, 2025

Quality Gate Passed Quality Gate passed for 'pypowsybl-parent'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@2nisb 2nisb mentioned this pull request Nov 20, 2025
Draft
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@2nisb 2nisb 2nisb left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@geofjamg @metab0t @2nisb @HugoKulesza