Skip to content

Conversation

mgovers
Copy link
Member

@mgovers mgovers commented Oct 10, 2025

Pandapower now has lxml as an optional dependency but it's actually not optional because to run import pandapower.networks, it also first needs to do (in pandapower.__init__.py) from pandapower.run_pp import *, which in turn imports pandapower.converter, which uses lxml. We do not want to have pandapower[converter] as a PGM dev dependency, as it also imports other things and it doesn't make sense to do so, so here we go.

Requires #333 before it can be merged to main.

@mgovers mgovers self-assigned this Oct 10, 2025
@mgovers mgovers added the dependencies Pull requests that update a dependency file label Oct 10, 2025
Signed-off-by: furqan463 <[email protected]>
nitbharambe
nitbharambe previously approved these changes Oct 13, 2025
mgovers and others added 5 commits October 13, 2025 09:13
Signed-off-by: Martijn Govers <[email protected]>
Signed-off-by: Martijn Govers <[email protected]>
Signed-off-by: furqan463 <[email protected]>
Pandapower converter: Fix compatibility issues with upcoming pandapower release.
Copy link

@mgovers mgovers added this pull request to the merge queue Oct 13, 2025
Merged via the queue into main with commit cbad605 Oct 13, 2025
31 checks passed
@mgovers mgovers deleted the feature/fix-nightly branch October 13, 2025 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants