Skip to content

Simplify cases_cepm file - #55

Merged
mdrewbeyer merged 7 commits into
devfrom
fix/sync-cases
Sep 29, 2026
Merged

mdrewbeyer merged 7 commits into
devfrom
fix/sync-cases

Conversation

@ty-fi

@ty-fi ty-fi commented Sep 29, 2026

Copy link
Copy Markdown
Collaborator

This PR simplifies cases_cepm and removes some of the outdated scenarios we had used for testing. It now has three geographies, set up in our two-step analysis format.

@ty-fi
ty-fi marked this pull request as ready for review September 29, 2026 15:54
@ty-fi
ty-fi requested a balanced review from Copilot September 29, 2026 16:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The SERTP cases select a different geography, and removed WECC-SW scenarios remain in operational documentation.

Review effort: Balanced
Findings: 2 Medium severity

Open (2)
What changed in this PR

Simplifies CEPM scenarios into three two-step analysis geographies.

Changes:

  • Replaces outdated scenarios with baseline, limited-RE, and optimized triplets.
  • Clarifies runtime capacity-cap configuration.
File Description
cases.csv Documents CEPM capacity-cap behavior.
cases_cepm.csv Defines the streamlined scenario matrix.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread cases_cepm.csv Outdated
Comment thread cases_cepm.csv
@ty-fi ty-fi assigned ty-fi and mdrewbeyer and unassigned ty-fi Sep 29, 2026
Updated the documentation to reflect changes in case columns and model names.
@github-actions github-actions Bot added the docs label Sep 29, 2026
Corrected 'st-MSALGA' to 'SERTP' in the documentation to ensure accuracy regarding baseline builds.
Updated the list of current cases and removed '_dcload' for both stems. Clarified the behavior of two-step columns and their interaction with the run_cepm.ps1 script.

@mdrewbeyer mdrewbeyer 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.

Replaced WECC-SW and SERTP with st-AZNM and st-MSALGA, respectively.

@mdrewbeyer
mdrewbeyer merged commit 5d36c64 into dev Sep 29, 2026
1 check passed
@mdrewbeyer
mdrewbeyer deleted the fix/sync-cases branch September 29, 2026 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants