NREL / OpenOA

This library provides a framework for assessing wind plant performance using operational assessment (OA) methodologies that consume time series data from wind plants. The goal of the project is to provide an open source implementation of common data structures, analysis methods, and utility functions relevant to wind plant OA.
https://openoa.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
193 stars 63 forks source link

Enhancement/logging review #254

Closed RHammond2 closed 1 year ago

RHammond2 commented 1 year ago

This PR adds logging back to OpenOA inPlantData and the analysis classes. To reduce visual clutter in analysis pipelines, a new log_level keyword argument has been added to PlantData to control which messages are displayed.

codecov-commenter commented 1 year ago

Codecov Report

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

Comparison is base (83ef14f) 65.28% compared to head (5467bc0) 65.75%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop_v3 #254 +/- ## ============================================== + Coverage 65.28% 65.75% +0.46% ============================================== Files 29 29 Lines 4197 4263 +66 ============================================== + Hits 2740 2803 +63 - Misses 1457 1460 +3 ``` | [Files](https://app.codecov.io/gh/NREL/OpenOA/pull/254?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL) | Coverage Δ | | |---|---|---| | [openoa/\_\_init\_\_.py](https://app.codecov.io/gh/NREL/OpenOA/pull/254?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-b3Blbm9hL19faW5pdF9fLnB5) | `100.00% <ø> (ø)` | | | [openoa/analysis/aep.py](https://app.codecov.io/gh/NREL/OpenOA/pull/254?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-b3Blbm9hL2FuYWx5c2lzL2FlcC5weQ==) | `84.81% <100.00%> (+0.03%)` | :arrow_up: | | [openoa/analysis/electrical\_losses.py](https://app.codecov.io/gh/NREL/OpenOA/pull/254?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-b3Blbm9hL2FuYWx5c2lzL2VsZWN0cmljYWxfbG9zc2VzLnB5) | `77.24% <100.00%> (+0.15%)` | :arrow_up: | | [openoa/analysis/eya\_gap\_analysis.py](https://app.codecov.io/gh/NREL/OpenOA/pull/254?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-b3Blbm9hL2FuYWx5c2lzL2V5YV9nYXBfYW5hbHlzaXMucHk=) | `95.16% <100.00%> (+0.16%)` | :arrow_up: | | [openoa/analysis/turbine\_long\_term\_gross\_energy.py](https://app.codecov.io/gh/NREL/OpenOA/pull/254?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-b3Blbm9hL2FuYWx5c2lzL3R1cmJpbmVfbG9uZ190ZXJtX2dyb3NzX2VuZXJneS5weQ==) | `76.32% <100.00%> (+0.69%)` | :arrow_up: | | [openoa/analysis/yaw\_misalignment.py](https://app.codecov.io/gh/NREL/OpenOA/pull/254?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-b3Blbm9hL2FuYWx5c2lzL3lhd19taXNhbGlnbm1lbnQucHk=) | `84.57% <100.00%> (+0.07%)` | :arrow_up: | | [openoa/logging.py](https://app.codecov.io/gh/NREL/OpenOA/pull/254?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-b3Blbm9hL2xvZ2dpbmcucHk=) | `70.96% <73.33%> (+3.11%)` | :arrow_up: | | [openoa/plant.py](https://app.codecov.io/gh/NREL/OpenOA/pull/254?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-b3Blbm9hL3BsYW50LnB5) | `75.31% <85.15%> (+1.62%)` | :arrow_up: |

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