Skip to content

Add transformer tap setting correction to match PSSE #1428

Merged
jd-lara merged 11 commits into
psy5from
mb/transformer-tap-correction
Jun 27, 2025
Merged

Add transformer tap setting correction to match PSSE #1428
jd-lara merged 11 commits into
psy5from
mb/transformer-tap-correction

Conversation

@m-bossart

Copy link
Copy Markdown
Contributor

Closes #1427

@m-bossart m-bossart self-assigned this Jun 23, 2025
@m-bossart

Copy link
Copy Markdown
Contributor Author

Need to do the same for Transformer3W and develop a test system with "bad" tap settings for both TapTransformer and Transformer3W

@jd-lara jd-lara requested a review from Copilot June 24, 2025 13:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the transformer tap setting calculation to match the PSSE specifications and adds a test to verify the correction logic.

  • Updated transformer tap calculation with a nearest-step correction logic in the PSSE parser.
  • Added a new test set for validating the transformer tap setting correction.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
test/test_parse_psse.jl Added test set for transformer tap position correction.
src/parsers/pm_io/psse.jl Updated transformer tap calculation with step correction logic.

Comment thread src/parsers/pm_io/psse.jl Outdated
@jd-lara jd-lara self-requested a review June 25, 2025 20:06
@m-bossart m-bossart marked this pull request as ready for review June 26, 2025 16:40
Comment thread src/parsers/pm_io/psse.jl
@m-bossart m-bossart requested a review from jd-lara June 27, 2025 19:43
@jd-lara jd-lara merged commit 6cd7f49 into psy5 Jun 27, 2025
1 of 9 checks passed
@jd-lara jd-lara deleted the mb/transformer-tap-correction branch August 1, 2025 21:48
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.

4 participants