Skip to content

Add .state.json files for PhasorDynamics - #643

Open
lukelowry wants to merge 2 commits into
developfrom
lukel/state-files-cases
Open

lukelowry wants to merge 2 commits into
developfrom
lukel/state-files-cases

Conversation

@lukelowry

@lukelowry lukelowry commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Adds initial state files for 17 PhasorDynamics cases, following the format introduced in #514 and the existing Hawaii example.

Proposed changes

  • Add .state.json files
  • Include bus voltages, device terminal currents, and operating settings.
  • Copy and install the state files alongside the corresponding examples

Checklist

  • JSON parsing and CMake copy/install checks pass.
  • N/A Code compiles cleanly with flags -Wall -Wpedantic -Wconversion -Wextra.
  • N/A The new code follows GridKit™ style guidelines.
  • N/A There are unit tests for the new code.
  • The file format is documented in STATE.md.
  • The feature branch is rebased with respect to the target branch.
  • N/A The CHANGELOG.md has been updated.

Further comments

State-file loading follows in a separate PR. This is the last 'preparation' needed before I can start migrating the .case.json initialization to this file. Then we unlock OPF

@lukelowry
lukelowry requested a review from nkoukpaizan November 3, 2026 20:11
@lukelowry
lukelowry marked this pull request as ready for review November 3, 2026 20:11
@lukelowry
lukelowry force-pushed the lukel/state-files-cases branch from e63b99a to c3e7526 Compare November 3, 2026 20:15

@pelesh pelesh left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks like a straightforward change. Good to merge.

@nkoukpaizan nkoukpaizan added the enhancement New feature or request label Nov 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants