search

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

Fix bug when adding StaticTimeSeries after Forecast #254

Closed daniel-thom closed 2 years ago

daniel-thom commented 2 years ago

There was a bug in the consistency checks that caused an exception to be thrown if a StaticTimeSeries was added after a Forecast.

codecov[bot] commented 2 years ago

Codecov Report

Merging #254 (7dec7f8) into master (d34c68d) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #254   +/-   ##
=======================================
  Coverage   76.85%   76.85%           
=======================================
  Files          43       43           
  Lines        3241     3241           
=======================================
  Hits         2491     2491           
  Misses        750      750
Flag Coverage Δ
unittests 76.85% <100.00%> (ø)

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

Impacted Files Coverage Δ
src/time_series_parameters.jl 94.89% <100.00%> (+0.05%) :arrow_up:
src/utils/lazy_dict_from_iterator.jl 96.15% <0.00%> (-0.15%) :arrow_down: