Skip to content

ACTIVSg2000, ACTIVSg10k, Hawaii, IEEE39 Validation Data - #564

Open
lukelowry wants to merge 6 commits into
developfrom
lukel/cases-polish-dev
Open

ACTIVSg2000, ACTIVSg10k, Hawaii, IEEE39 Validation Data#564
lukelowry wants to merge 6 commits into
developfrom
lukel/cases-polish-dev

Conversation

@lukelowry

Copy link
Copy Markdown
Collaborator

Description

Validation data (and updated cases with supported models) for ACTIVSg2000, ACTIVSg10k, Hawaii, IEEE39.

Proposed changes

  • Added validation data for four monitored series for each case
  • Added updated cases to cases/ in clean, maintainable format
  • One-lines for each case

Checklist

  • All tests pass.
  • Code compiles cleanly with flags -Wall -Wpedantic -Wconversion -Wextra.
  • The new code follows GridKit™ style guidelines.
  • There are unit tests for the new code.
  • The new code is documented.
  • The feature branch is rebased with respect to the target branch.
  • The CHANGELOG.md has been updated to reflect the changes. If this is a minor PR that is part of a larger fix already included in the file, state so.

Further comments

Major success. PowerWorld doesn't expose controls to lower the network solution below 1e-5 tolerance, so I think we are nearing the tightest validation error we can get with the PowerWorld reference.
ACTIVSg2000 omega

IEEE39 omega WECC240 omega ACTIVSg10k omega

@lukelowry
lukelowry force-pushed the lukel/cases-polish-dev branch from 2651008 to 2cf81d3 Compare September 1, 2026 16:39
@lukelowry
lukelowry marked this pull request as ready for review September 1, 2026 16:44
@lukelowry

Copy link
Copy Markdown
Collaborator Author

Once this is merged, we should be able to remove the older cases from examples/PhasorDynamics/ so that the examples just consume the cases through *.solver.json

@nkoukpaizan nkoukpaizan added new case New Power System Case with GridKit Input Format new study New study or event capability labels Sep 3, 2026
@lukelowry
lukelowry force-pushed the lukel/cases-polish-dev branch from 2cf81d3 to d443851 Compare September 3, 2026 16:01
@lukelowry lukelowry mentioned this pull request Sep 4, 2026
7 tasks
@lukelowry
lukelowry force-pushed the lukel/cases-polish-dev branch 2 times, most recently from b97ed93 to f16f381 Compare September 5, 2026 17:09
@lukelowry

Copy link
Copy Markdown
Collaborator Author

@nkoukpaizan This one is ready!

Comment on lines +188 to +191
parser = argparse.ArgumentParser()
parser.add_argument("study", type=Path)
parser.add_argument("--rundir", type=Path)
parser.add_argument("--outdir", type=Path)

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.

I'm debating whether to include this python code or not. At a minimum, we'd want some usage documentation and error handling for incorrect arguments.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Your call, I threw it in just so the style of the plots stays consistent. I don't have a preference

@lukelowry
lukelowry force-pushed the lukel/cases-polish-dev branch from f16f381 to f799123 Compare September 8, 2026 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new case New Power System Case with GridKit Input Format new study New study or event capability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants