Skip to content

Distinction of Magnetization vs. Line Shunt (Validation)#486

Draft
lukelowry wants to merge 1 commit into
developfrom
lukel/mag-shunt-dev
Draft

Distinction of Magnetization vs. Line Shunt (Validation)#486
lukelowry wants to merge 1 commit into
developfrom
lukel/mag-shunt-dev

Conversation

@lukelowry

@lukelowry lukelowry commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Description

Implementation on develop treats the shunt as a magnetizing admittance, which should have a distinct parameter from the line shunt admittance, which does not observe the effects of phase/tap changes. MATPOWER does not seem to make this distinction, but PowerWorld models this correctly. This contributed marginally to errors in magnetization admittances in some cases.

Required before 0.2 release.

Proposed changes

Describe how your changes here address the issue and why the proposed changes
should be accepted.

Checklist

Put an x in the boxes that apply. You can also fill these out after creating
the PR. If you're unsure about any of them, don't hesitate to ask. We're here
to help! This is simply a reminder of what we are going to look for before
merging your code.

  • 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.
  • I have updated CHANGELOG.md to reflect the changes in this PR. If this is a minor PR that is part of a larger fix already included in the file, state so.

Further comments

Need to update the circuit mdoel diagram

@lukelowry lukelowry requested a review from abirchfield July 10, 2026 23:46
@lukelowry lukelowry force-pushed the lukel/mag-shunt-dev branch from 260ec5e to 8513b61 Compare July 11, 2026 00:41
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.

1 participant