Skip to content

Merge links into nodes: update pf and sc tests - #1569

Merged
Jerry-Jinfeng-Guo merged 4 commits into
feature/link-registration-dual-code-pathfrom
feature/link-tests-update-after-merging
Sep 4, 2026
Merged

Merge links into nodes: update pf and sc tests #1569
Jerry-Jinfeng-Guo merged 4 commits into
feature/link-registration-dual-code-pathfrom
feature/link-tests-update-after-merging

Conversation

@Jerry-Jinfeng-Guo

@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo commented Sep 4, 2026

Copy link
Copy Markdown
Member

This PR provides tests that are updated according to new links calculation results. Some differ from #1507 due to calculation by hand.

Short circuit calculation test case for line-to-itself edge case is updated. To be followed-up in #1566.

Signed-off-by: Jerry Jinfeng Guo <jerry.jinfeng.guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo self-assigned this Sep 4, 2026
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo added the feature New feature or request label Sep 4, 2026
Signed-off-by: Jerry Jinfeng Guo <jerry.jinfeng.guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo changed the title update pf results and mark sc tests as fail Merge links into nodes: update pf results and mark sc tests as fail Sep 4, 2026
Comment thread tests/data/short_circuit/dummy-test-line-into-itself/params.json Outdated
Signed-off-by: Jerry Jinfeng Guo <jerry.jinfeng.guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo changed the title Merge links into nodes: update pf results and mark sc tests as fail Merge links into nodes: update pf and sc tests Sep 4, 2026
Signed-off-by: Jerry Jinfeng Guo <jerry.jinfeng.guo@alliander.com>
@Jerry-Jinfeng-Guo
Jerry-Jinfeng-Guo merged commit 25934a4 into feature/link-registration-dual-code-path Sep 4, 2026
9 of 32 checks passed
@Jerry-Jinfeng-Guo
Jerry-Jinfeng-Guo deleted the feature/link-tests-update-after-merging branch September 4, 2026 12:46
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.

2 participants