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

coszen made not optional in Dudhia swrad #1675

Merged
merged 1 commit into from
Mar 15, 2022

Conversation

dudhia
Copy link
Collaborator

@dudhia dudhia commented Feb 15, 2022

TYPE: bug fix
KEYWORDS: shortwave option 1, coszen not optional

SOURCE: internal, reported by CWB, Taiwan

DESCRIPTION OF CHANGES:
Problem:
Compiler picked up use of uninitialized eot. This was in the case of no coszen passed in which does not happen in ARW.

Solution:
coszen is no longer optional and branch for no coszen input is removed.

LIST OF MODIFIED FILES:
M phys/module_ra_sw.F

TESTS CONDUCTED:

  1. Only compile test
  2. The Jenkins tests are all passing.

RELEASE NOTE:

@dudhia dudhia requested a review from a team as a code owner February 15, 2022 17:13
@weiwangncar weiwangncar merged commit 179fd33 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: shortwave option 1, coszen not optional

SOURCE: internal, reported by CWB, Taiwan

DESCRIPTION OF CHANGES:
Problem:
Compiler picked up use of uninitialized eot. This was in the case of no coszen passed in which does not happen in ARW.

Solution:
coszen is no longer optional and branch for no coszen input is removed.

LIST OF MODIFIED FILES:
M phys/module_ra_sw.F

TESTS CONDUCTED:
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