E3SM-Project / E3SM

Energy Exascale Earth System Model source code. NOTE: use "maint" branches for your work. Head of master is not validated.
https://docs.e3sm.org/E3SM
Other
346 stars 353 forks source link

CAM4 fails to build in F case on Titan/PGI #227

Closed worleyph closed 9 years ago

worleyph commented 9 years ago

Today this builds successfully:

./create_newcase -case F1850C5.ne30 -mach titan -compset F1850C5 -res ne30_g16 -project cli115 -compiler pgi

while this does not

./create_newcase -case F1850.ne30 -mach titan -compset F1850 -res ne30_g16 -project cli115 -compiler pgi

The error message is PGF90-F-0004-Unable to open MODULE file modal_aero_convproc.mod (/autofs/na3_home1/worley/ACPI/SVN/ACME/master/ACME/models/atm/cam/src/physics/cam/physpkg.F90: 1747)

and, in fact, I do not see a module file called modal_aero_convproc being built. (It is not that it is failing to build - it is simply has not been built before it is referenced.)

This is CAM4, so perhaps we do not care about it. Atmosphere Group should advise, and also verify on other systems, though this seems to be a build logic issue not a system-specific issue.

Note: Problem was first identified by @bmayerornl (for a T31_g37 FVCAM build). So this issue occurs for multiple dycores and multiple resolutions.

mt5555 commented 9 years ago

This was caused when the mam4 code was merged in. Balwinder is working on a fix. (CAM5 prescribed aerosols is also broken in the same way)

CAM4 is not tested in ACME, and my be dropped by the atmosphere team

singhbalwinder commented 9 years ago

I am re-factoring the code to resolve this issue. I will move most of the unified convection changes to the src/chemistry/modal_aero folder so that they don't interfere with other model configurations (such as prescribed aerosols, CAM4 etc.).

worleyph commented 9 years ago

Thanks for the explanation and for the update.

philrasch commented 9 years ago

I do not believe we should be testing or maintaining CAM4 code any longer. ACME has no use for it. If someone else wants to make a case for keeping it I am open to doing so, but I don't see any point.