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

Fix a logic for using specific humidity or mixing ratio directly #2031

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

weiwangncar
Copy link
Collaborator

@weiwangncar weiwangncar commented Mar 26, 2024

TYPE: bug fix

KEYWORDS: real, use_sh_qv

SOURCE: internal

DESCRIPTION OF CHANGES:
The logic to control whether specific humidity or mixing ratio is used instead of RH is wrong (PR-1959). It is corrected in this PR.

LIST OF MODIFIED FILES:
M dyn_em/module_initialize_real.F

TESTS CONDUCTED:

  1. Tested before and after this change. The data without SH is not affected, and the data with SH (EC model level data) is now used directly if use_sh_qv is set to true.
  2. The Jenkins tests are all passing.

RELEASE NOTE:

@mgduda mgduda changed the base branch from master to release-v4.6.0 March 26, 2024 03:26
@weiwangncar
Copy link
Collaborator Author

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

@dudhia
Copy link
Collaborator

dudhia commented Mar 26, 2024

So you don't need to test on those flags?

@weiwangncar
Copy link
Collaborator Author

So you don't need to test on those flags?

No.

@weiwangncar
Copy link
Collaborator Author

@kkeene44 @islas Can you review and approve this PR?

@kkeene44
Copy link
Collaborator

@weiwangncar
I'm just curious what happened before the fix. Would something fail, down the line?

@weiwangncar
Copy link
Collaborator Author

@weiwangncar I'm just curious what happened before the fix. Would something fail, down the line?

real.exe would not fail. When the input has one of the SH or Qv input, it doesn't matter if one sets use_sh_qv or not, it will always use computed RH (if RH isn't in the input. If RH is in the input, it uses it too).

@weiwangncar weiwangncar merged commit 0b68af0 into wrf-model:release-v4.6.0 Mar 27, 2024
1 of 15 checks passed
@dudhia
Copy link
Collaborator

dudhia commented Mar 27, 2024

Does this fix anything in the last release or is it just fixing things added since?

@weiwangncar
Copy link
Collaborator Author

@dudhia This change is only added for this release.

islas pushed a commit to islas/WRF that referenced this pull request Jun 10, 2024
…rf-model#2031)

TYPE: bug fix

KEYWORDS: real, use_sh_qv

SOURCE: internal

DESCRIPTION OF CHANGES:
The logic to control whether specific humidity or mixing ratio is used instead of RH is wrong (PR-1959) when either SH or Qv is available. It is corrected in this PR.

LIST OF MODIFIED FILES:
M dyn_em/module_initialize_real.F

TESTS CONDUCTED:

Tested before and after this change. The data without SH is not affected, and the data with SH (EC model level data) is now used directly if use_sh_qv is set to true.
The Jenkins tests are all passing.
islas pushed a commit to islas/WRF that referenced this pull request Jun 10, 2024
…rf-model#2031)

TYPE: bug fix

KEYWORDS: real, use_sh_qv

SOURCE: internal

DESCRIPTION OF CHANGES:
The logic to control whether specific humidity or mixing ratio is used instead of RH is wrong (PR-1959) when either SH or Qv is available. It is corrected in this PR.

LIST OF MODIFIED FILES:
M dyn_em/module_initialize_real.F

TESTS CONDUCTED:

Tested before and after this change. The data without SH is not affected, and the data with SH (EC model level data) is now used directly if use_sh_qv is set to true.
The Jenkins tests are all passing.
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.

None yet

3 participants