Skip to content

Composite Load Model Parameter Export Expansion - #61

Open
HaleyRoss wants to merge 2 commits into
masterfrom
hr/update_cmpld_exports
Open

Composite Load Model Parameter Export Expansion#61
HaleyRoss wants to merge 2 commits into
masterfrom
hr/update_cmpld_exports

Conversation

@HaleyRoss

Copy link
Copy Markdown
Collaborator

Add to the list of parameters from the PSS/E composite load model that can be exported.

@HaleyRoss
HaleyRoss requested review from nadiavp and a lite review from Copilot August 4, 2026 21:26

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.

🟢 Ready to approve

The new composite load parameters are added consistently across constants, enums, and default export settings without introducing broken references or invalid mappings.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

This PR expands PyPSSE’s exportable composite load model parameters by adding additional dynamic composite load coefficients (P/Q corner points) to the internal constants, public enumerations, and the default export configuration, enabling these parameters to be included in exports.

Changes:

  • Added P1c, P2c, Q1c, Q2c to the dynamic load model parameter index map (lmodind) and dynamic-only property list.
  • Extended LoadProperties enum to include the new composite load parameters.
  • Updated default export_settings.toml so load exports include the new parameters by default.
File summaries
File Description
pypsse/modes/constants.py Adds P1c/P2c/Q1c/Q2c to dynamic-only load parameter indices and dynamic-only property mapping.
pypsse/enumerations.py Exposes the new load properties via the LoadProperties enum for validation and API usage.
pypsse/defaults/export_settings.toml Updates default load export property list to include the newly supported parameters.
Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 0
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

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.

2 participants