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

Updated heat transfer coefficient correlation parameters for horizontal and upward heat flow over PV panels. #2049

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

joshi994
Copy link
Contributor

@joshi994 joshi994 commented May 14, 2024

TYPE: Bug fix

KEYWORDS: PV panels, Correlations, Upward heat flow, BEM (Building Energy Model).

SOURCE: Parag Joshi (Brookhaven National Laboratory)

DESCRIPTION OF CHANGES:
Problem:
A typographical mistake was identified within the correlations that evaluate heat transfer coefficient for the upward heat flow over PV panels in the Building Energy Model (BEM).

Solution:
The parameters have been modified following the correlation expressions provided in the literature. Please refer to the equation J.2.2a (please see the attached screenshot) on page number 79 of the document below.
https://nvlpubs.nist.gov/nistpubs/Legacy/IR/nbsir83-2655.pdf

Screenshot 2024-05-14 at 5 07 59 PM

LIST OF MODIFIED FILES:
M phys/module_sf_bem.F

TESTS CONDUCTED:

  1. No test has been conducted as the parameters seems to have very small or insignificant impact on the results. However, the tests can be performed with and without correcting the parameters to quantify the role.
  2. The Jenkins tests are all passing.

RELEASE NOTE: Bug fixes for parameters associated with the heat transfer coefficient for the upward heat flow over PV panels.

…coefficient for the upward/downward transport of heat are corrected.
@joshi994 joshi994 requested a review from a team as a code owner May 14, 2024 21:10
@weiwangncar
Copy link
Collaborator

The regression test results:
Test Type | Expected | Received | Failed
= = = = = = = = = = = = = = = = = = = = = = = = = = = =
Number of Tests : 23 24
Number of Builds : 60 57
Number of Simulations : 158 150 0
Number of Comparisons : 95 86 0

Failed Simulations are: 
None
Which comparisons are not bit-for-bit: 
None

@weiwangncar
Copy link
Collaborator

@joshi994 Please hold off any update to this PR or any new PR, as we are going to update the regression tests in the next few weeks. Thanks!

@dudhia
Copy link
Collaborator

dudhia commented May 16, 2024

This PR is to master and should go to develop until we have a 4.6.1 branch

@islas islas changed the base branch from master to release-v4.6.1 May 16, 2024 20:10
@weiwangncar
Copy link
Collaborator

@cenlinhe Can you review this PR?

@cenlinhe
Copy link
Contributor

cenlinhe commented Aug 6, 2024

While I am reviewing this, I would also like the original developer @andreazonato to take a look and confirm this bug fix.

@weiwangncar
Copy link
Collaborator

@andreazonato Can you review this PR? Thanks!

@andreazonato
Copy link
Contributor

Sorry for the late reply, I was on holidays.

Yes, I think this PR solves the bug.

I agree! Thanks.

Andrea

@cenlinhe
Copy link
Contributor

I approve this fix.

@weiwangncar weiwangncar merged commit 6273c88 into wrf-model:release-v4.6.1 Aug 20, 2024
1 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants