search

NREL-Sienna / PowerSystems.jl

Data structures in Julia to enable power systems analysis. Part of the Scalable Integrated Infrastructure Planning Initiative at the National Renewable Energy Lab.
https://www.nrel.gov/analysis/sienna.html
BSD 3-Clause "New" or "Revised" License
306 stars 76 forks source link

Kdrh/psy4 docs renaming #1141

Closed jd-lara closed 3 months ago

GabrielKS commented 3 months ago

Context: this PR relating to the parser code was merged, but it was merged into another feature branch that did not subsequently make it into main for some reason.

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 88.57143% with 8 lines in your changes missing coverage. Please review.

Project coverage is 84.60%. Comparing base (e6f1161) to head (c40ec43). Report is 1 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/NREL-Sienna/PowerSystems.jl/pull/1141/graphs/tree.svg?width=650&height=150&src=pr&token=AEzJsDhpxL&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna)](https://app.codecov.io/gh/NREL-Sienna/PowerSystems.jl/pull/1141?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 #1141 +/- ## ========================================== - Coverage 84.60% 84.60% -0.01% ========================================== Files 177 177 Lines 8083 8108 +25 ========================================== + Hits 6839 6860 +21 - Misses 1244 1248 +4 ``` | [Flag](https://app.codecov.io/gh/NREL-Sienna/PowerSystems.jl/pull/1141/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/PowerSystems.jl/pull/1141/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna) | `84.60% <88.57%> (-0.01%)` | :arrow_down: | 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](https://app.codecov.io/gh/NREL-Sienna/PowerSystems.jl/pull/1141?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna) | Coverage Δ | | |---|---|---| | [src/parsers/power\_models\_data.jl](https://app.codecov.io/gh/NREL-Sienna/PowerSystems.jl/pull/1141?src=pr&el=tree&filepath=src%2Fparsers%2Fpower_models_data.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL3BhcnNlcnMvcG93ZXJfbW9kZWxzX2RhdGEuamw=) | `91.94% <100.00%> (ø)` | | | [src/parsers/power\_system\_table\_data.jl](https://app.codecov.io/gh/NREL-Sienna/PowerSystems.jl/pull/1141?src=pr&el=tree&filepath=src%2Fparsers%2Fpower_system_table_data.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL3BhcnNlcnMvcG93ZXJfc3lzdGVtX3RhYmxlX2RhdGEuamw=) | `81.90% <88.40%> (-0.39%)` | :arrow_down: | ... and [3 files with indirect coverage changes](https://app.codecov.io/gh/NREL-Sienna/PowerSystems.jl/pull/1141/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna)
jd-lara commented 3 months ago

The PSB tests are expect to fail due to the new implementation of the CDM parser.