EarthWorksOrg / EarthWorks

Other
3 stars 3 forks source link

New cam_dev files cause non-simple physics compsets to fail #55

Closed gdicker1 closed 2 months ago

gdicker1 commented 5 months ago

Real-data IC and topo files added in PRs EWOrg/CAM #16 and EWOrg/CAM #20 cause run-time fails of compsets that use "full physics" - especially those with chemistry and radiation interactions.

"Simple-physics" compsets like FADIAB and any MPAS-A standalone runs are successful. (NOTE: some compsets like FHS94 are also unaffected since they use separate "notopo" files.)

Compsets affected: F2000climo (when requesting these files), F2000dev, and CHAOS2000dev.

Example case on Derecho: /glade/derecho/scratch/gdicker/tstgrids_F2000dev_2_mpasa480_20240606-110547/run

How to reproduce:

  1. Clone the most recent release or the development branch of EarthWorks
  2. Create a compset that uses full physics and mpas grid (e.g. ./create_newcase --compset F2000dev --res mpasa480_mpasa480 )
  3. Submit the case after setting up and building

Symptoms seen:

Sample output seen in atm.log.* and cesm.log.*

 imp_sol: Time step   1.1250000000000E+01 failed to converge @ (lchnk,lev,col,nstep) =      2    32     4     0
 imp_sol: Failed to converge @ (lchnk,lev,col,nstep,dt,time) =      2    32     4     0  1.1250000000000E+01  1.7887500000000E+03
 num_a1    1.000E+00
 so4_a1    1.000E+00
 imp_sol: Time step   1.1250000000000E+01 failed to converge @ (lchnk,lev,col,nstep) =      2    32     4     0
 imp_sol: Failed to converge @ (lchnk,lev,col,nstep,dt,time) =      2    32     4     0  1.1250000000000E+01  1.8000000000000E+03
 num_a1    1.000E+00
 so4_a1    1.000E+00
 imp_sol : @ (lchnk,lev,col) =            2          32           4  failed          165  times

Workaround

While I continue to investigate, this issue can be mitigated by just not using these files. Supply paths to the relevant files in the ncdata and bnd_topo files in a case's user_nl_cam file. If you wish to use cam_dev physics in a case with the old files, you will need to supply use_gw_front = .false. in the user_nl_cam file.

adamrher commented 4 months ago

This is a shot in the dark, but Francis found a bug cam_dev where the liquid and ice paths being fed into radiation aren't being initialized. Maybe try his changes to physpkg.F90 and cloud_diagnostics.F90 in https://github.com/ESCOMP/CAM/pull/1074 ?

gdicker1 commented 3 months ago

The changes in https://github.com/EarthWorksOrg/CAM/pull/23 will address this issue. It is waiting on an upstream version of the PR in ESCOMP/CAM (#1095) before being merged into EarthWorks and the files in https://github.com/EarthWorksOrg/CAM/pull/16 and https://github.com/EarthWorksOrg/CAM/pull/20 will be added again.

gdicker1 commented 2 months ago

Resolved by merge of #73. Please revive this issue or start a new one if problems are noticed.