Skip to content

Soiling algorithm updates Quyen's branch#432

Closed
qnguyen345 wants to merge 37 commits into
NatLabRockies:developmentfrom
qnguyen345:bare_except_error
Closed

Soiling algorithm updates Quyen's branch#432
qnguyen345 wants to merge 37 commits into
NatLabRockies:developmentfrom
qnguyen345:bare_except_error

Conversation

@qnguyen345

@qnguyen345 qnguyen345 commented Oct 24, 2024

Copy link
Copy Markdown
  • 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
Signed-off-by: nmoyer <noah.moyer@nrel.gov>
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

codecov-commenter commented Oct 24, 2024

Copy link
Copy Markdown

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
Copy Markdown
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 NatLabRockies: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
Copy Markdown
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