Conversation
|
The tolerance seems too high I need to make sure the tests are correct before review |
Performance ResultsPrecompile Time
Execution TimeCells show median (min–max) over 5 samples; delta compares medians.
|
|
@jd-lara the tests are passing locally for me with main of PowerSystemCaseBuilder. This is ready for review |
There was a problem hiding this comment.
Pull request overview
This PR adds support for applying PSS/E Impedance Correction Data (ICD/ICT) when constructing the AC Ybus, and updates the PSS/E regression tests + fixtures to validate the corrected transformer admittances (including off-nominal cases).
Changes:
- Compute ICD correction factors from
PSY.ImpedanceCorrectionDatacurves and apply them by scaling transformer series impedance during Ybus assembly. - Precompute a per-transformer correction map and thread the correction factor through branch iteration into
_ybus!/ybus_branch_entries. - Update existing PSS/E Ybus comparisons to use Float32-relative tolerances and add two new PSS/E fixture matrices for ICD-enabled/disabled modified 14-bus cases.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
src/Ybus.jl |
Adds ICD curve interpolation + correction-factor plumbing into transformer Ybus contributions, plus new tests-facing assembly paths (currently with a broken adj plumbing that needs fixes). |
src/YbusACBranches.jl |
Introduces ICDCorrectionMap and extends branch iteration to supply a per-branch correction factor. |
test/test_ybus_psse.jl |
Tightens numeric comparisons to Float32-relative tolerances and adds new testsets covering off-nominal and nominal ICD cases. |
test/test_data/modified_14bus_system_off_nominal_icd_ymatrix.txt |
New PSS/E Y-matrix fixture for the off-nominal ICD case. |
test/test_data/modified_14bus_system_no_icd.txt |
New PSS/E Y-matrix fixture for the nominal/no-ICD case. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| tb, | ||
| ix, | ||
| network_reduction_data, | ||
| adj, | ||
| c1, |
|
@jd-lara this is ready for review. The tests pass locally with |
Requires: