search

NREL-Sienna / PowerNetworkMatrices.jl

Methods to generate matrix representations of power systems matrices
BSD 3-Clause "New" or "Revised" License
20 stars 8 forks source link

modified PTDF functions for Dense method #58

Closed alefcastelli closed 1 year ago

alefcastelli commented 1 year ago

Modified methods for the Dense method. Significant improvements in computational time were recorded

github-actions[bot] commented 1 year ago
Performance Results Version Precompile Time
Main 2.547323677
This Branch 2.590044415
Version Execute Time
Main-Build PTDF First 5.131222607
Main-Build PTDF Second 0.126453405
Main-Build Ybus First 0.424831178
Main-Build Ybus Second 0.010313635
Main-Build LODF First 0.605555054
Main-Build LODF Second 0.260098748
This Branch-Build PTDF First 5.640806902
This Branch-Build PTDF Second 0.160099344
This Branch-Build Ybus First 0.425639845
This Branch-Build Ybus Second 0.00946495
This Branch-Build LODF First 0.558417946
This Branch-Build LODF Second 0.254845944
codecov[bot] commented 1 year ago

Codecov Report

Merging #58 (8ae8a0d) into main (14ad47f) will increase coverage by 0.23%. The diff coverage is 100.00%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/NREL-Sienna/PowerNetworkMatrices.jl/pull/58/graphs/tree.svg?width=650&height=150&src=pr&token=2VvekKsf11&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna)](https://app.codecov.io/gh/NREL-Sienna/PowerNetworkMatrices.jl/pull/58?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna) ```diff @@ Coverage Diff @@ ## main #58 +/- ## ========================================== + Coverage 78.14% 78.37% +0.23% ========================================== Files 14 14 Lines 1130 1119 -11 ========================================== - Hits 883 877 -6 + Misses 247 242 -5 ``` | [Flag](https://app.codecov.io/gh/NREL-Sienna/PowerNetworkMatrices.jl/pull/58/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/NREL-Sienna/PowerNetworkMatrices.jl/pull/58/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna) | `78.37% <100.00%> (+0.23%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files Changed](https://app.codecov.io/gh/NREL-Sienna/PowerNetworkMatrices.jl/pull/58?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna) | Coverage Δ | | |---|---|---| | [src/ptdf\_calculations.jl](https://app.codecov.io/gh/NREL-Sienna/PowerNetworkMatrices.jl/pull/58?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL3B0ZGZfY2FsY3VsYXRpb25zLmps) | `94.70% <100.00%> (+2.77%)` | :arrow_up: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/NREL-Sienna/PowerNetworkMatrices.jl/pull/58/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna)