WISDEM / WOMBAT

Windfarm Operations & Maintenance cost-Benefit Analysis Tool
https://wisdem.github.io/WOMBAT/
Apache License 2.0
21 stars 11 forks source link

Bug Fix: Only Process Weather Delays Once #81

Closed RHammond2 closed 1 year ago

RHammond2 commented 1 year ago

This PR fixes erroneous logic causing weather delays to be processed twice due to improper control flow.

Closes #80

codecov[bot] commented 1 year ago

Codecov Report

Merging #81 (c83506a) into develop (8644f87) will decrease coverage by 0.16%. The diff coverage is 66.66%.

@@             Coverage Diff             @@
##           develop      #81      +/-   ##
===========================================
- Coverage    56.09%   55.94%   -0.16%     
===========================================
  Files           21       21              
  Lines         3271     3273       +2     
  Branches       617      618       +1     
===========================================
- Hits          1835     1831       -4     
- Misses        1348     1351       +3     
- Partials        88       91       +3     
Impacted Files Coverage Δ
wombat/core/service_equipment.py 62.76% <66.66%> (-0.77%) :arrow_down:
wombat/core/repair_management.py 73.61% <0.00%> (-0.62%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.