NREL-Sienna / InfrastructureSystems.jl

Utility package for Sienna's simulation infrastructure
https://nrel-sienna.github.io/InfrastructureSystems.jl/
BSD 3-Clause "New" or "Revised" License
39 stars 21 forks source link

Use proper julia argument order in get_components #301

Closed daniel-thom closed 1 year ago

daniel-thom commented 1 year ago

Fixes #300

codecov[bot] commented 1 year ago

Codecov Report

Merging #301 (42bfda3) into master (5c71c34) will increase coverage by 0.25%. The diff coverage is 100.00%.

Additional details and impacted files [![Impacted file tree graph](https://codecov.io/gh/NREL-SIIP/InfrastructureSystems.jl/pull/301/graphs/tree.svg?width=650&height=150&src=pr&token=I73yjxYxgn&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-SIIP)](https://codecov.io/gh/NREL-SIIP/InfrastructureSystems.jl/pull/301?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-SIIP) ```diff @@ Coverage Diff @@ ## master #301 +/- ## ========================================== + Coverage 78.05% 78.30% +0.25% ========================================== Files 44 44 Lines 3672 3660 -12 ========================================== Hits 2866 2866 + Misses 806 794 -12 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `78.30% <100.00%> (+0.25%)` | :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-SIIP#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/NREL-SIIP/InfrastructureSystems.jl/pull/301?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-SIIP) | Coverage Δ | | |---|---|---| | [src/system\_data.jl](https://codecov.io/gh/NREL-SIIP/InfrastructureSystems.jl/pull/301/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-SIIP#diff-c3JjL3N5c3RlbV9kYXRhLmps) | `87.50% <100.00%> (+0.08%)` | :arrow_up: | | [src/InfrastructureSystems.jl](https://codecov.io/gh/NREL-SIIP/InfrastructureSystems.jl/pull/301/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-SIIP#diff-c3JjL0luZnJhc3RydWN0dXJlU3lzdGVtcy5qbA==) | `77.77% <0.00%> (-2.23%)` | :arrow_down: | | [src/utils/utils.jl](https://codecov.io/gh/NREL-SIIP/InfrastructureSystems.jl/pull/301/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-SIIP#diff-c3JjL3V0aWxzL3V0aWxzLmps) | `62.25% <0.00%> (-0.39%)` | :arrow_down: | | [src/hdf5\_time\_series\_storage.jl](https://codecov.io/gh/NREL-SIIP/InfrastructureSystems.jl/pull/301/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-SIIP#diff-c3JjL2hkZjVfdGltZV9zZXJpZXNfc3RvcmFnZS5qbA==) | `94.04% <0.00%> (+2.93%)` | :arrow_up: |