-
Notifications
You must be signed in to change notification settings - Fork 20
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
Initialize gaussian_data in gaussian_sfcanl #71
Comments
I approve this solution. Thanks for catching my bug. |
@GeorgeGayno-NOAA: Here is a RUNDIR for my C48 test on Hera: /scratch1/NCEPDEV/da/Catherine.Thomas/git/global-workflow/c48test/TMP/RUNDIRS/c48test/analcalc.410225 Let me know if you need me to run anything else. |
Case set up on Hera: /scratch1/NCEPDEV/da/George.Gayno/global-workflow.git/gaussian_sfcanl |
undefined value. Fixes NOAA-EMC#71.
Started another test branch. Added some compiler options (576a55c) that will initialize variables to a very large value that will hopefully cause a run time failure. The test case was rerun using the new branch. A floating overflow was detected:
|
All variables were initialized to zero (289d090) and the test case was re-run. It completed normally:
|
What is wrong?
In the gaussian_sfcanl utility, the variable
gaussian_data
is used before it is initialized to zero (Line 273):Sometimes the resulting values are unreasonable (for example 1E+200).
What should have happened?
The
gaussian_data
variable should be initialized to zero after allocation.What machines are impacted?
All or N/A
Steps to reproduce
veg
Additional information
More details are in global-workflow issue #2669.
Do you have a proposed solution?
The
gaussian_data
variable should be initialized to zero after allocation.The text was updated successfully, but these errors were encountered: