NREL / rdtools

PV Analysis Tools in Python
https://rdtools.readthedocs.io/
MIT License
158 stars 67 forks source link

RdTools 3.0 #390

Closed mdeceglie closed 1 month ago

mdeceglie commented 1 year ago
mdeceglie commented 1 year ago

Very early draft for now

codecov-commenter commented 7 months ago

Codecov Report

Attention: Patch coverage is 93.30629% with 66 lines in your changes missing coverage. Please review.

Please upload report for BASE (development@996f843). Learn more about missing BASE report.

Files with missing lines Patch % Lines
rdtools/soiling.py 90.41% 48 Missing :warning:
rdtools/analysis_chains.py 93.67% 11 Missing :warning:
rdtools/plotting.py 90.62% 3 Missing :warning:
rdtools/degradation.py 94.87% 2 Missing :warning:
rdtools/filtering.py 98.89% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## development #390 +/- ## ============================================== Coverage ? 96.10% ============================================== Files ? 11 Lines ? 2209 Branches ? 0 ============================================== Hits ? 2123 Misses ? 86 Partials ? 0 ```

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

mdeceglie commented 1 month ago

A couple more changes based on our internal discussions. If this still looks good to everyone, I will update the changelog with 3.0beta, add a date and get this merged to development so we can polish of the soiling changes ahead of the full release.

martin-springer commented 1 month ago

A couple more changes based on our internal discussions. If this still looks good to everyone, I will update the changelog with 3.0beta, add a date and get this merged to development so we can polish of the soiling changes ahead of the full release.

Looks good to me!