WISDEM / WOMBAT

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

Bug Fix: Correct Intra-Site Travel #84

Closed RHammond2 closed 1 year ago

RHammond2 commented 1 year ago

This PR fixes a bug where ServiceEquipment.crew_transfer() was changing the location from a specific system to the site when a crew was transferred back to the servicing equipment causing the intra-site travel to be from None to a system instead of from a system to another system with regards to distance traveled.

codecov[bot] commented 1 year ago

Codecov Report

Merging #84 (d830351) into develop (d3acd76) will increase coverage by 0.36%. The diff coverage is 30.55%.

@@             Coverage Diff             @@
##           develop      #84      +/-   ##
===========================================
+ Coverage    55.98%   56.35%   +0.36%     
===========================================
  Files           21       21              
  Lines         3281     3267      -14     
  Branches       619      614       -5     
===========================================
+ Hits          1837     1841       +4     
+ Misses        1352     1336      -16     
+ Partials        92       90       -2     
Impacted Files Coverage Δ
wombat/core/mixins.py 64.70% <ø> (ø)
wombat/core/port.py 17.85% <0.00%> (+0.73%) :arrow_up:
wombat/core/simulation_api.py 50.92% <0.00%> (ø)
wombat/core/repair_management.py 76.25% <37.50%> (+2.63%) :arrow_up:
wombat/core/service_equipment.py 63.70% <83.33%> (+0.93%) :arrow_up:

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