Add RRM CONUS case configuration updates to reduce new user edits #168

eroesler · 2015-04-01T22:32:31Z

These additions allow the user to create the CONUS RRM case in a more out-of-the-box way without having to enter namelist values in the user_nl* files, the grid location, or edit env_run.xml. Two namelist parameters were not in build.namelist when the RRM case configuration for the CONUS grid was first developed and recently pushed to master. This update required a change to namelist_mod.F90 so that when using structured grids, a grid file is not needed.

It should be noted that the tuning paramters (i.e. zmconv_tau, etc) have not been tuned for the CONUS grid but have been transferred from ne30 and armx8v3 configurations.

Machine files for Sandia computers were also updated.

NML

AG-120

Two namelist parameters were not in build.namelist when the RRM case configuration was developeed. These additions allow the user to create an RRM case out-of-the-box without having to enter namelist values in the user_nl* files. On branch eroesler/ATM/RRM-meshfilefixer Changes to be committed: modified: build-namelist

These changes allow a user to build the variable resolution CONUS RRM ACME configuration out-of-the-box without having to specify the values for some physics paramterizations in the user_nl_* files. WARNING: As of this commit, the values have not been tuned for the CONUS grid but are values used to tune either the ne30 or armx8v3 grid. On branch eroesler/ATM/RRM-meshfilefixer Changes to be committed: modified: namelist_files/namelist_defaults_cam.xml

Three sets of changes in this commit. 1. The namelist_defaults_cam.xml and the namelist_mod.F90 have updates so that a user wanting to run RRM case does not have to specify the path of the grid file in the user_nl_cam. This will reduce potential error and improve ease of the RRM capability. It was found that the mesh_file designation needed to be blank for a structured (non-RRM) grid, so namelist_mod.F90 was altered to not have a model crash during runtime. 2. The paramter EPS_AAREA needed to be raised in env_run.xml by a factor of 10. This was done by hand every time an RRM case was built. The changes to config_grid.xml now have the correct value of EPS_AAREA to run RRM. 3. Final changes is to $PROJECT variable for SNL computers. On branch eroesler/ATM/RRM-meshfilefixer Changes to be committed: modified: ../models/atm/cam/bld/namelist_files/namelist_defaults_cam.xml modified: ../models/atm/cam/src/dynamics/se/share/namelist_mod.F90 modified: ccsm_utils/Case.template/config_grid.xml modified: ccsm_utils/Machines/mkbatch.redsky modified: ccsm_utils/Machines/mkbatch.skybridge

eroesler · 2015-04-01T22:33:30Z

acme_developer on skybridge passed.

mt5555 · 2015-04-01T22:46:11Z

Can you remove all those physics tunings from this pull request? For RRM grids, we should use the default values for all those parameters, unless somoene has taken the time to tune them.

In particular, I think we should not set special values for the RRM grid for these parameters:

soil_erod_file, dust_emis_fact, cldfrc_rhminl, cldfrc_dp1, zmconv_c0_lnd, zmconv_c0_ocn, zmconv_tau

Also, I think there is a problem with zmconv_tau: this pull request looks like it might set a default value of 3000 for all simulations. I'm pretty sure this is supposed to be 3600, not 3000.

eroesler · 2015-04-02T15:23:14Z

Hi @mt5555,

I'd like to take the time to tune those parameters for CONUS, and I'll fix the zmconv_tau. Thanks for catching that.

The parameter zmconv_tau was incorrectly set to 3000. It is now set to 3600. On branch eroesler/ATM/RRM-meshfilefixer Changes to be committed: modified: ../models/atm/cam/bld/namelist_files/namelist_defaults_cam.xml

yjinho · 2015-04-05T06:46:36Z

Ok then, I'll hold it. @eroesler, please let me know when it is ready.

Default values will be used unless specified by the user. These parameters have been removed : soil_erod_file, dust_emis_fact, cldfrc_rhminl, cldfrc_dp1, zmconv_c0_lnd, zmconv_c0_ocn, zmconv_tau On branch eroesler/ATM/RRM-meshfilefixer Changes to be committed: modified: models/atm/cam/bld/namelist_files/namelist_defaults_cam.xml modified: models/atm/cam/src/dynamics/se/share/namelist_mod.F90 modified: scripts/ccsm_utils/Case.template/config_grid.xml

Forgot to delete comments for RRM grids in last commit, so fixed text in this commit. On branch eroesler/ATM/RRM-meshfilefixer Changes to be committed: modified: models/atm/cam/bld/namelist_files/namelist_defaults_cam.xml

eroesler · 2015-04-06T16:43:35Z

Hi @yjinho and @mt5555 (and @susburrows),

The settings for the tuning parameters are removed so that the RRM grid will now run with the defaults. I've retested with these changes and have rerun the acme_developer's tests, which all passed.

The branch should be ready to be merged.

yjinho · 2015-04-06T16:45:58Z

Thanks @eroesler !! I'll get this moving.

These additions allow the user to create the CONUS RRM case in a more out-of-the-box way without having to enter namelist values in the user_nl* files, the grid location, or edit env_run.xml. Two namelist parameters were not in build.namelist when the RRM case configuration for the CONUS grid was first developed and recently pushed to master. This update required a change to namelist_mod.F90 so that when using structured grids, a grid file is not needed. It should be noted that the tuning paramters (i.e. zmconv_tau, etc) have not been tuned for the CONUS grid but have been transferred from ne30 and armx8v3 configurations. Machine files for Sandia computers were also updated. The settings for the tuning parameters are removed so that the RRM grid will now run with the defaults. I've retested with these changes and have rerun the acme_developer's tests, which all passed. [NML] [AG-120] Conflicts: models/atm/cam/bld/namelist_files/namelist_defaults_cam.xml

yjinho · 2015-04-07T17:27:24Z

It is now in next (done after midnight). So, I'll wait another day to see the tests and to merge this to master.

jgfouca · 2015-04-09T16:51:18Z

It broke the tests.

jgfouca · 2015-04-09T16:51:42Z

scripts/ccsm_utils/Machines/mkbatch.redsky

@@ -55,7 +55,7 @@ cat >! $file << EOF1
 #===============================================================================
 #SBATCH --job-name ${jobname}
 #SBATCH -N ${nodes}
-#SBATCH --account=${PROJECT}
+#SBATCH --account=FY${PROJECT}


Breaks a bunch of stuff

…and skybridge Broke the nightly testing since other scripts were not aware of this magic. Probably simpler to just always require the full project. [BFB]

This is the second merge to next for this PR. Problems were found in the nightly testing that needed to be fixed requiring additional commits on this branch. * eroesler/ATM/RRM-meshfilefixer: Remove automatic appending of FY to job submission account on redsky and skybridge

jgfouca · 2015-04-09T17:17:07Z

OK, the project issue has been fixed, added to this branch, and merged to next.

eroesler · 2015-04-14T17:22:10Z

Hi @yjinho, When do you think this branch will go to master?

yjinho · 2015-04-14T18:04:33Z

Hi @eroesler, I'm doing it now. @jgfouca has to fix one thing and I waited it a couple of more days.

eroesler · 2015-04-14T18:06:26Z

Hi @yjinho -- sounds good! Thanks!

These additions allow the user to create the CONUS RRM case in a more out-of-the-box way without having to enter namelist values in the user_nl* files, the grid location, or edit env_run.xml. Two namelist parameters were not in build.namelist when the RRM case configuration for the CONUS grid was first developed and recently pushed to master. This update required a change to namelist_mod.F90 so that when using structured grids, a grid file is not needed. It should be noted that the tuning paramters (i.e. zmconv_tau, etc) have not been tuned for the CONUS grid but have been transferred from ne30 and armx8v3 configurations. Machine files for Sandia computers were also updated. The settings for the tuning parameters are removed so that the RRM grid will now run with the defaults. I've retested with these changes and have rerun the acme_developer's tests, which all passed. Also it fixes automatic appending of FY to job on redsky and skybridge. [BFB] [NML] [AG-120] Conflicts: models/atm/cam/bld/namelist_files/namelist_defaults_cam.xml

…-integration Pacer integration into Omega standalone driver

eroesler added 3 commits March 30, 2015 10:29

eroesler added Atmosphere Machine Files labels Apr 1, 2015

eroesler assigned yjinho Apr 1, 2015

Changed zmconv_tau from 3000 to 3600

2c9a362

The parameter zmconv_tau was incorrectly set to 3000. It is now set to 3600. On branch eroesler/ATM/RRM-meshfilefixer Changes to be committed: modified: ../models/atm/cam/bld/namelist_files/namelist_defaults_cam.xml

eroesler added 2 commits April 6, 2015 10:29

Fixing typo in namelist_defaults_cam.xml

db516bf

Forgot to delete comments for RRM grids in last commit, so fixed text in this commit. On branch eroesler/ATM/RRM-meshfilefixer Changes to be committed: modified: models/atm/cam/bld/namelist_files/namelist_defaults_cam.xml

jgfouca reviewed Apr 9, 2015
View reviewed changes

Remove automatic appending of FY to job submission account on redsky …

d71e80f

…and skybridge Broke the nightly testing since other scripts were not aware of this magic. Probably simpler to just always require the full project. [BFB]

susburrows mentioned this pull request Apr 9, 2015

Implement short-term time step convergence test. #181

Merged

yjinho merged commit d71e80f into master Apr 14, 2015

jgfouca deleted the eroesler/ATM/RRM-meshfilefixer branch October 22, 2015 18:34

philipwjones added a commit to philipwjones/E3SM that referenced this pull request Dec 4, 2024

Merge pull request E3SM-Project#168 from climatemodeling/sarats/pacer…

8f008c7

…-integration Pacer integration into Omega standalone driver

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add RRM CONUS case configuration updates to reduce new user edits #168

Add RRM CONUS case configuration updates to reduce new user edits #168

eroesler commented Apr 1, 2015

eroesler commented Apr 1, 2015

mt5555 commented Apr 1, 2015

eroesler commented Apr 2, 2015

yjinho commented Apr 5, 2015

eroesler commented Apr 6, 2015

yjinho commented Apr 6, 2015

yjinho commented Apr 7, 2015

jgfouca commented Apr 9, 2015

jgfouca Apr 9, 2015

jgfouca commented Apr 9, 2015

eroesler commented Apr 14, 2015

yjinho commented Apr 14, 2015

eroesler commented Apr 14, 2015

Add RRM CONUS case configuration updates to reduce new user edits #168

Add RRM CONUS case configuration updates to reduce new user edits #168

Conversation

eroesler commented Apr 1, 2015

eroesler commented Apr 1, 2015

mt5555 commented Apr 1, 2015

eroesler commented Apr 2, 2015

yjinho commented Apr 5, 2015

eroesler commented Apr 6, 2015

yjinho commented Apr 6, 2015

yjinho commented Apr 7, 2015

jgfouca commented Apr 9, 2015

jgfouca Apr 9, 2015

Choose a reason for hiding this comment

jgfouca commented Apr 9, 2015

eroesler commented Apr 14, 2015

yjinho commented Apr 14, 2015

eroesler commented Apr 14, 2015