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

Allow override of time series directory with environment variable #252

Closed daniel-thom closed 2 years ago

daniel-thom commented 2 years ago

There is a minor mess that causes CI to fail. Now this PR and https://github.com/NREL-SIIP/PowerSystems.jl/pull/793 are dependent on each other.

codecov[bot] commented 2 years ago

Codecov Report

Merging #252 (fde9861) into master (c2e7689) will increase coverage by 0.16%. The diff coverage is 88.09%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #252      +/-   ##
==========================================
+ Coverage   76.42%   76.59%   +0.16%     
==========================================
  Files          43       43              
  Lines        3165     3205      +40     
==========================================
+ Hits         2419     2455      +36     
- Misses        746      750       +4     
Flag Coverage Δ
unittests 76.59% <88.09%> (+0.16%) :arrow_up:

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

Impacted Files Coverage Δ
src/time_series_storage.jl 77.77% <ø> (ø)
src/in_memory_time_series_storage.jl 73.10% <71.42%> (+0.46%) :arrow_up:
src/hdf5_time_series_storage.jl 93.54% <92.30%> (-0.05%) :arrow_down:
src/component.jl 94.00% <100.00%> (+0.30%) :arrow_up:
src/internal.jl 78.00% <100.00%> (+0.44%) :arrow_up:
src/system_data.jl 86.66% <100.00%> (+0.11%) :arrow_up:
src/utils/lazy_dict_from_iterator.jl 96.15% <0.00%> (-0.15%) :arrow_down: