PowerSimulations.jl icon indicating copy to clipboard operation
PowerSimulations.jl copied to clipboard

Sd/cost fix

Open jd-lara opened this issue 3 years ago • 1 comments

jd-lara avatar Sep 21 '22 20:09 jd-lara

Codecov Report

Merging #882 (15fa894) into master (52539c8) will decrease coverage by 0.04%. The diff coverage is 69.69%.

:exclamation: Current head 15fa894 differs from pull request most recent head 3d9a7b6. Consider uploading reports for the commit 3d9a7b6 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #882      +/-   ##
==========================================
- Coverage   85.31%   85.27%   -0.05%     
==========================================
  Files         117      117              
  Lines       12586    12603      +17     
==========================================
+ Hits        10738    10747       +9     
- Misses       1848     1856       +8     
Flag Coverage Δ
unittests 85.27% <69.69%> (-0.05%) :arrow_down:

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

Impacted Files Coverage Δ
src/operation/emulation_model.jl 87.20% <0.00%> (-1.03%) :arrow_down:
src/core/optimization_container.jl 79.01% <66.66%> (-0.63%) :arrow_down:
src/operation/decision_model.jl 95.71% <100.00%> (+0.06%) :arrow_up:
src/services_models/services_constructor.jl 98.57% <100.00%> (+0.01%) :arrow_up:
src/utils/dataframes_utils.jl 42.85% <100.00%> (+3.46%) :arrow_up:

codecov[bot] avatar Sep 21 '22 21:09 codecov[bot]

@sourabhdalvi I think this MR needs to be format fixed.

jd-lara avatar Oct 17 '22 19:10 jd-lara