-
Notifications
You must be signed in to change notification settings - Fork 150
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
part of cam6_4_011: fix the path to fms for fv3 build, remove mct reference #1067
Conversation
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.
Looks good. Thanks for cleaning this up.
@jedwards4b, I'm confused. We have an several FV3 tests in the aux_cam regression tests, and they all are passing. I just tried the test referred to in #1068 (ERS_Ln9.C96_C96_mg17.FHS94.derecho_intel.cam-outfrq9s) on izumi/gnu (since derecho is down) and it builds and runs. I tested using the latest cam tag cam6_4_005. The FMS.bldlog file does contain a warning about a missing include file, but it's not an error. Does this PR just clean that up, or is it more than that? |
It's more than that - the cime PR ESMCI/cime#4647 changes the path to the FMS library removing "nuopc" at line 55 and comp_interface at line 111. Since the current cam externals don't have 4647 your tests will still pass without this change. |
@jedwards4b, I tried pulling in this PR along with the new cime6.1.0 tag and removing the mct submodule. My first build attempt fails with the error (in csm_share.bldlog)
The file is in |
You need the new share tag as well: share1.1.2 |
@jedwards4b, Do I need to update the cmeps tag as well? Currently using
|
Yes cmeps1.0.0. |
The build is now completing, but a test of F2000climo at f10 with debug is getting a seg fault:
Any ideas? |
I do not get this error using SMS_D_Ln9.f19_f19_mg17.F2000climo.derecho_intel.cam-outfrq9s or with /scratch/cluster/jedwards/SMS_D_Ln9.f10_f10_mg37.F2000climo.izumi_intel.cam-outfrq9s.20240708_104012_g3o8ng |
I ran Start from cam6_4_006, merge jedwards4b:fix/fms_path, update submodules to cime6.1.0, cmeps1.0.0, and share1.1.2, and remove mct submodule. This source is available from my CAM fork on branch |
I started over, followed your instructions, built the following test - still passes: |
Thanks for looking at this. I ran the test on izumi and got the same result as you. But izumi has really old gnu compilers, v9.3.0. I'm using v11.4.0 on my desktop. So I reran on derecho which has newer gnu compilers, v12.2.0 and v13.2.0 (not sure what the cime tests are configured to use). On derecho I get what appears to be the same failure I got on my desktop:
Since the cmeps code has been updated I'm suspecting the problem was introduced there. |
Fixed in ESCOMP/CMEPS#479 |
Thanks @jedwards4b. FYI, it turns out that NAG debug also caught this problem. |
To be clear - this is really a non-bug bug report. The compilers are reporting an unallocated or unassociated variable in a subroutine interface for a variable that is never actually used. |
The path to the FMS library needs to be updated for fv3 builds.
Fixes issue #1068