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
35 stars 20 forks source link

Re-design time series management #349

Closed daniel-thom closed 2 months ago

daniel-thom commented 2 months ago

New features:

Refactor/re-design:

Features removed:

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 92.61905% with 62 lines in your changes are missing coverage. Please review.

Project coverage is 75.61%. Comparing base (335a5a5) to head (5c89df2). Report is 3 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349/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-Sienna)](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?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 #349 +/- ## ========================================== + Coverage 74.37% 75.61% +1.24% ========================================== Files 64 68 +4 Lines 4952 4876 -76 ========================================== + Hits 3683 3687 +4 + Misses 1269 1189 -80 ``` | [Flag](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349/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/InfrastructureSystems.jl/pull/349/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna) | `75.61% <92.61%> (+1.24%)` | :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](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?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/InfrastructureSystems.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2FInfrastructureSystems.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL0luZnJhc3RydWN0dXJlU3lzdGVtcy5qbA==) | `80.00% <ø> (ø)` | | | [src/abstract\_time\_series.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2Fabstract_time_series.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL2Fic3RyYWN0X3RpbWVfc2VyaWVzLmps) | `85.71% <100.00%> (+10.71%)` | :arrow_up: | | [src/component.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2Fcomponent.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL2NvbXBvbmVudC5qbA==) | `91.66% <100.00%> (-3.60%)` | :arrow_down: | | [src/containers.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2Fcontainers.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL2NvbnRhaW5lcnMuamw=) | `100.00% <100.00%> (ø)` | | | [src/deterministic\_metadata.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2Fdeterministic_metadata.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL2RldGVybWluaXN0aWNfbWV0YWRhdGEuamw=) | `95.45% <100.00%> (+86.36%)` | :arrow_up: | | [src/probabilistic.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2Fprobabilistic.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL3Byb2JhYmlsaXN0aWMuamw=) | `81.66% <100.00%> (ø)` | | | [src/scenarios.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2Fscenarios.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL3NjZW5hcmlvcy5qbA==) | `85.18% <100.00%> (ø)` | | | [src/serialization.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2Fserialization.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL3NlcmlhbGl6YXRpb24uamw=) | `70.33% <ø> (+9.70%)` | :arrow_up: | | [src/single\_time\_series.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2Fsingle_time_series.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL3NpbmdsZV90aW1lX3Nlcmllcy5qbA==) | `68.00% <100.00%> (ø)` | | | [src/supplemental\_attribute.jl](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree&filepath=src%2Fsupplemental_attribute.jl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna#diff-c3JjL3N1cHBsZW1lbnRhbF9hdHRyaWJ1dGUuamw=) | `94.44% <100.00%> (+13.59%)` | :arrow_up: | | ... and [22 more](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna) | | ... and [3 files with indirect coverage changes](https://app.codecov.io/gh/NREL-Sienna/InfrastructureSystems.jl/pull/349/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL-Sienna)