Skip to content

[codex] Add state estimation interface#270

Draft
skyswordw wants to merge 1 commit into
PowerGridModel:mainfrom
skyswordw:codex/add-state-estimation-interface
Draft

[codex] Add state estimation interface#270
skyswordw wants to merge 1 commit into
PowerGridModel:mainfrom
skyswordw:codex/add-state-estimation-interface

Conversation

@skyswordw
Copy link
Copy Markdown

Summary

  • add PowerGridModelInterface.calculate_state_estimation() as a thin wrapper around PowerGridModel.calculate_state_estimation()
  • store and return state-estimation output in output_data, matching the existing power-flow interface pattern
  • add an integration test using a simple grid with voltage and power sensors

Closes #20

Validation

  • uv run pytest tests/integration/loadflow/test_power_grid_model.py -q
  • uv run ruff format --check src/power_grid_model_ds/_core/power_grid_model_interface.py tests/integration/loadflow/test_power_grid_model.py
  • uv run ruff check src/power_grid_model_ds/_core/power_grid_model_interface.py tests/integration/loadflow/test_power_grid_model.py
  • git diff --check
  • uv run python -m compileall -q src/power_grid_model_ds tests/integration/loadflow/test_power_grid_model.py
  • uv run pytest -q

Signed-off-by: skywalker <skywalkerwang98@gmail.com>
@skyswordw skyswordw force-pushed the codex/add-state-estimation-interface branch from 84b06f4 to 9e2fa0b Compare June 4, 2026 16:04
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.

[FEATURE] Add support for calculate_state_estimation

1 participant