Skip to content

changed DHI calculation to not zero out DNI when dhi is calculated as…#12

Merged
bnb32 merged 1 commit intomainfrom
gb/fix_dhi
Jun 28, 2024
Merged

changed DHI calculation to not zero out DNI when dhi is calculated as…#12
bnb32 merged 1 commit intomainfrom
gb/fix_dhi

Conversation

@grantbuster
Copy link
Copy Markdown
Member

… negative

@grantbuster grantbuster requested review from bnb32 and xieyupku June 27, 2024 21:57
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 27, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 57.68%. Comparing base (c154171) to head (e17e79e).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   57.32%   57.68%   +0.36%     
==========================================
  Files           9        9              
  Lines         478      475       -3     
==========================================
  Hits          274      274              
+ Misses        204      201       -3     
Flag Coverage Δ
unittests 57.68% <0.00%> (+0.36%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
farms/utilities.py 23.30% <0.00%> (+0.65%) ⬆️

@bnb32 bnb32 merged commit c5a9076 into main Jun 28, 2024
@bnb32 bnb32 deleted the gb/fix_dhi branch June 28, 2024 20:26
github-actions bot pushed a commit that referenced this pull request Jun 28, 2024
changed DHI calculation to not zero out DNI when dhi is calculated as…
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.

3 participants