WISDEM / WOMBAT

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

OpEx Breakdown Function #119

Closed dmulas closed 1 year ago

dmulas commented 1 year ago

Add function to output:

by frequency type, and the average by frequency type

RHammond2 commented 1 year ago

It looks like this is nearly there, we just need two things:

  1. An updated in the CHANGELOG to call out the modification
  2. To ensure the formatting is all consistent by running the pre-commit hooks in your wombat environment

    cd /path/to/WOMBAT/  # if you're not already 
    pre-commit install
    pre-commit run --files wombat/core/post_processor.py
    
    # These steps might need to be run more than once depending on if they pass,
    # require manual correction (unlikely), or have corrections automatically applied
    git add wombat/core/post_processor.py  # the changes need to be registered
    git commit -m "your message"  # you'll notice that pre-commit runs again at this phase
    
    # Once all the commit checks pass or get skipped
    git push
codecov[bot] commented 1 year ago

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (20fcb13) 53.76% compared to head (b800a39) 53.73%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #119 +/- ## =========================================== - Coverage 53.76% 53.73% -0.03% =========================================== Files 22 22 Lines 3722 3724 +2 Branches 765 766 +1 =========================================== Hits 2001 2001 - Misses 1599 1601 +2 Partials 122 122 ``` | [Files](https://app.codecov.io/gh/WISDEM/WOMBAT/pull/119?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=WISDEM) | Coverage Δ | | |---|---|---| | [wombat/core/post\_processor.py](https://app.codecov.io/gh/WISDEM/WOMBAT/pull/119?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=WISDEM#diff-d29tYmF0L2NvcmUvcG9zdF9wcm9jZXNzb3IucHk=) | `5.28% <33.33%> (-0.02%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.