-
Notifications
You must be signed in to change notification settings - Fork 21
Bring in nco changes for DEBUG intel flags, bump GOCART to latest develop #342
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
Conversation
mathomp4
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Easy approve as these don't affect GEOS
|
@BrianCurtis-NOAA Do you still want this in? |
|
I do. I've had a few front-burner items that have pushed this to the back. As soon as I can coordinate this effort across all UFSWM components, I'll move it off draft-mode. Apologies to keep this waiting. |
No problem. My fear was this was meant to be not-draft long ago and was forgotten, etc. It doesn't affect us so we should be able to take it at anytime. |
|
Will the latest develop changes cause baseline differences? I'm seeing some, and wanted to verify that any tests using GOCART will have results changing. |
@BrianCurtis-NOAA GOCART https://github.com/GEOS-ESM/GOCART/compare/main...develop?w=1 You can see that a pressure lid is now used in many places (say 40 mbar in black carbon). The Ginoux scheme and other scalings also had updates for L181 which we are moving to. We also updated some optics files. Finally, I'll mention @vbuchard @pcolarco and @jrjoshi1 for more edification. |
|
@mathomp4 We'll need to look at restart reproducibility with the latest GOCART in UFSWM. We had to ditch it. My changes will not change results, so it's still OK to be in GEOS-ESM/develop. |
@BrianCurtis-NOAA We just didn't want to lose this PR of yours. We are almost done with updating is the last "big" one we are looking at. So if you want to test the "future" |
Bring in NCO DEBUG flag change for ops
The following flags will be added for DEBUG builds: -check all -check noarg_temp_created -ftrapuv
Two are required by NCO: -check all -ftrapuv
One will drastically reduce build output file size: -check noarg_temp_created
I've tested the build on the UFSWM. I will run the full UFSWM RT suite soon to ensure the build change does not impact results. I do not anticipate any changes.