Skip to content

[CHORE] Remove dead legacy converter edisgo/tools/powermodels_io.py #699

Description

@joda9

Details

edisgo/tools/powermodels_io.py is a leftover pypower-based PowerModels converter. It is
imported nowhere in the code base (no imports in edisgo/, tests/, or examples/) — the
active converter used by the OPF pipeline is edisgo/io/powermodels_io.py.

Why remove it (beyond dead weight)

The file is actively misleading: it contains storage-efficiency constants (e.g. 0.9487) and
conversion logic that differ from what the OPF actually uses. When investigating OPF behaviour
(e.g. the storage-efficiency bug, see #694), grepping for powermodels_io surfaces
this file first and suggests parameters that never reach the Julia side.

Suggested fix

Delete edisgo/tools/powermodels_io.py (and its test file, if any). If parts are ever needed
again, they remain available in git history.

Metadata

Metadata

Assignees

Labels

good first issueWell-scoped, low-risk — suitable for newcomers / student assistantshousekeeping

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions