Skip to content

Step-up transformer tap changer support: feature branch #826

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 128 commits into from
Mar 14, 2025

Conversation

Jerry-Jinfeng-Guo
Copy link
Contributor

@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo commented Nov 11, 2024

This is the feature branch for step-up transformer tap changer support. Closes #662.

Dedicated tests:

Feature additions (w/ unit tests):

@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo added feature New feature or request do-not-merge This should not be merged labels Nov 11, 2024
Jerry-Jinfeng-Guo and others added 20 commits November 11, 2024 09:42
Signed-off-by: Nitish Bharambe <nitish.bharambe@alliander.com>
Signed-off-by: Nitish Bharambe <nitish.bharambe@alliander.com>
…validation

Feature / add unique regulators condition to data validator
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
…er-ranking

Feature / Updated transformer weight before transformer ranking
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Copy link
Member

@mgovers mgovers left a comment

Choose a reason for hiding this comment

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

a couple minor suggestions after all PRs have been reviewd, but nothing major. Also feel free to resolve without applying if you disagree.

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Jerry-Jinfeng-Guo and others added 3 commits March 11, 2025 16:49
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <6221579+Jerry-Jinfeng-Guo@users.noreply.github.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
figueroa1395
figueroa1395 previously approved these changes Mar 12, 2025
Copy link
Contributor

@figueroa1395 figueroa1395 left a comment

Choose a reason for hiding this comment

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

There are no further comments from me. Hereby I give my approval.

mgovers
mgovers previously approved these changes Mar 13, 2025
Copy link
Member

@mgovers mgovers left a comment

Choose a reason for hiding this comment

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

only one more readability comment. Otherwise approved.

…ptimizer/tap_position_optimizer.hpp

Co-authored-by: Martijn Govers <martijn.govers@alliander.com>
Signed-off-by: Jerry Guo <6221579+Jerry-Jinfeng-Guo@users.noreply.github.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo dismissed stale reviews from mgovers and figueroa1395 via e911275 March 13, 2025 13:30
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo added this pull request to the merge queue Mar 14, 2025
Merged via the queue into main with commit ac4ecf9 Mar 14, 2025
28 checks passed
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo deleted the feature/step-up-transformer-tap-changer-support branch March 14, 2025 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Automatic tap changer in step-up transformer support
6 participants