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

resolve uninitialized variable issues in surface_driver #1681

Merged
merged 1 commit into from
Mar 15, 2022

Conversation

dudhia
Copy link
Collaborator

@dudhia dudhia commented Feb 17, 2022

TYPE: bug-fix
KEYWORDS: surface driver, uninitialized variables

SOURCE: internal, reported by CWB, Taiwan

DESCRIPTION OF CHANGES:
Problem:
Several variables showed up as used while uninitialized
Solution:
PXLSM does not need to compute sea-ice weighted PBLH
sfclay,sfclayrev FH and FM need to be initialized for use in sea-ice weighting

LIST OF MODIFIED FILES:
M phys/module_surface_driver.F

TESTS CONDUCTED:

  1. compile test only - OK
  2. The Jenkins tests are all passing.

RELEASE NOTE:

@dudhia dudhia requested a review from a team as a code owner February 17, 2022 20:53
@dudhia
Copy link
Collaborator Author

dudhia commented Feb 17, 2022

passes all tests
Test Type | Expected | Received | Failed
= = = = = = = = = = = = = = = = = = = = = = = = = = = =
Number of Tests : 23 24
Number of Builds : 60 58
Number of Simulations : 158 156 0
Number of Comparisons : 95 92 0

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

@dudhia dudhia changed the base branch from master to release-v4.4 February 17, 2022 22:05
@weiwangncar weiwangncar merged commit c394ff0 into wrf-model:release-v4.4 Mar 15, 2022
vlakshmanan-scala pushed a commit to scala-computing/WRF that referenced this pull request Apr 4, 2024
TYPE: bug-fix
KEYWORDS: surface driver, uninitialized variables

SOURCE: internal, reported by CWB, Taiwan

DESCRIPTION OF CHANGES:
Problem:
Several variables showed up as used while uninitialized

Solution:
PXLSM does not need to compute sea-ice weighted PBLH. FH and FM in seaice wrapper calls to sfclay, sfclayrev need to be initialized for use in sea-ice weighting.

LIST OF MODIFIED FILES:
M phys/module_surface_driver.F

TESTS CONDUCTED:
The Jenkins tests have 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.

3 participants