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

Fix exception in copy_time_series! #235

Closed daniel-thom closed 3 years ago

daniel-thom commented 3 years ago

There was a bug that caused a runtime exception when copy_time_series! was called after transform_single_time_series!.

Fixes #234

codecov[bot] commented 3 years ago

Codecov Report

Merging #235 (61a2722) into master (d503647) will increase coverage by 0.07%. The diff coverage is 92.30%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #235      +/-   ##
==========================================
+ Coverage   75.28%   75.36%   +0.07%     
==========================================
  Files          43       43              
  Lines        3115     3121       +6     
==========================================
+ Hits         2345     2352       +7     
+ Misses        770      769       -1     
Flag Coverage Δ
unittests 75.36% <92.30%> (+0.07%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/component.jl 93.69% <88.88%> (+0.10%) :arrow_up:
src/hdf5_time_series_storage.jl 93.39% <100.00%> (+0.01%) :arrow_up:
src/in_memory_time_series_storage.jl 72.38% <100.00%> (+0.53%) :arrow_up:
src/utils/utils.jl 61.18% <0.00%> (-0.43%) :arrow_down:
src/utils/lazy_dict_from_iterator.jl 96.15% <0.00%> (-0.15%) :arrow_down:
src/utils/logging.jl 89.78% <0.00%> (ø)
src/generated/DeterministicMetadata.jl 64.00% <0.00%> (+8.00%) :arrow_up: