Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Soiling algorithm updates Quyen's branch #432

Closed
wants to merge 37 commits into from

Conversation

qnguyen345
Copy link

@qnguyen345 qnguyen345 commented Oct 24, 2024

  • Code changes are covered by tests
  • Code changes have been evaluated for compatibility/integration with TrendAnalysis
  • New functions added to __init__.py
  • API.rst is up to date, along with other sphinx docs pages
  • Example notebooks are rerun and differences in results scrutinized
  • Updated changelog

nmoyer and others added 30 commits June 24, 2024 11:57
…tio and fit multiple soiling rates per soiling interval (piecewise)) as well as CODS algorithm being added
Move SRR and CODS development branch from noromo01 to rdtools repo
@qnguyen345 qnguyen345 changed the base branch from dev_SRR_CODS to aggregated_filters_for_trials October 24, 2024 21:03
@codecov-commenter
Copy link

codecov-commenter commented Oct 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.04%. Comparing base (16e23e6) to head (6172ea0).

Additional details and impacted files
@@                        Coverage Diff                        @@
##           aggregated_filters_for_trials     #432      +/-   ##
=================================================================
+ Coverage                          95.92%   96.04%   +0.11%     
=================================================================
  Files                                 11       11              
  Lines                               2209     2375     +166     
=================================================================
+ Hits                                2119     2281     +162     
- Misses                                90       94       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@martin-springer
Copy link
Collaborator

@mdeceglie - Quyen (@qnguyen345) finished the soiling PR. We haven't saved the 2 notebooks yet to give you a chance to look at the (minor) differences in the soiling results.

@mdeceglie mdeceglie deleted the branch NREL:development October 29, 2024 03:30
@mdeceglie mdeceglie closed this Oct 29, 2024
@mdeceglie mdeceglie reopened this Oct 30, 2024
@mdeceglie mdeceglie changed the base branch from aggregated_filters_for_trials to development October 30, 2024 15:06
@mdeceglie mdeceglie mentioned this pull request Nov 4, 2024
6 tasks
@mdeceglie mdeceglie changed the title Fixing Pylint Bare Except Error and Adding Pytests Soiling algorithm updates Nov 4, 2024
@martin-springer
Copy link
Collaborator

Closing in favor of PR #435.

@martin-springer martin-springer changed the base branch from development to qnguyen345-bare_except_error November 8, 2024 16:50
@martin-springer martin-springer changed the title Soiling algorithm updates Soiling algorithm updates Quyen's branch Nov 8, 2024
@martin-springer martin-springer changed the base branch from qnguyen345-bare_except_error to development November 8, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants