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: Tidy the plotting library #253

Closed RHammond2 closed 1 year ago

RHammond2 commented 1 year ago

This PR brings the entire openoa/utils/plot.py up to spec with later implemented best practices to ensure that figures can be customized in the methods to suit most needs for publication or presentation, or return the matplotlib objects back to add the final touches.

Given that plot_windfarm was added at a later point in time and is much cleaner, plot_array has been deprecated entirely. However, the following have been removed as a byproduct of their original intent being lost and their usage being unclear:

This PR closes #251.

codecov-commenter commented 1 year ago

Codecov Report

Attention: Patch coverage is 8.33333% with 22 lines in your changes missing coverage. Please review.

Project coverage is 65.50%. Comparing base (daf42ac) to head (371c4f2).

Files with missing lines Patch % Lines
openoa/utils/plot.py 8.33% 22 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop_v3 #253 +/- ## ============================================== + Coverage 61.85% 65.50% +3.64% ============================================== Files 29 29 Lines 4315 4061 -254 ============================================== - Hits 2669 2660 -9 + Misses 1646 1401 -245 ```

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