cam6_4_063: Update namelist settings #1252
Conversation
There was a problem hiding this comment.
Thanks @cacraigucar!
This is beyond the scope of the current PR, but I wonder if we should use the same values for MPAS as well for dust_emis_fact since it currently keys off the dycore with dyn="se" (at least as a starting point instead of the default)
In terms of code, add
<dust_emis_fact dyn="mpas" phys="cam7" >2.30D0</dust_emis_fact>
If @adamrher is OK with that I think we should make that change now to avoid possible wasted CPU hours running simulations thinking the paramters are the same as SE with MPAS ...
| @@ -2529,6 +2529,7 @@ | |||
| <dust_emis_fact dyn="fv" phys="cam5" clubb_sgs="1" >0.22D0</dust_emis_fact> | |||
| <dust_emis_fact dyn="se" phys="cam6" >0.70D0</dust_emis_fact> | |||
| <dust_emis_fact dyn="se" phys="cam7" >2.30D0</dust_emis_fact> | |||
| <dust_emis_fact dyn="mpas" phys="cam7" >2.30D0</dust_emis_fact> | |||
There was a problem hiding this comment.
What about dust_emis_fact for cam7 and fv?
There was a problem hiding this comment.
@PeterHjortLauritzen @adamrher - I need you to answer this
There was a problem hiding this comment.
There are numerous entries for fv and cam7, and so my last comment "all cam7 entries should be 2.3" would resolve this.
There was a problem hiding this comment.
We are not supporting FV with CAM7
| @@ -2529,6 +2529,7 @@ | |||
| <dust_emis_fact dyn="fv" phys="cam5" clubb_sgs="1" >0.22D0</dust_emis_fact> | |||
| <dust_emis_fact dyn="se" phys="cam6" >0.70D0</dust_emis_fact> | |||
| <dust_emis_fact dyn="se" phys="cam7" >2.30D0</dust_emis_fact> | |||
| <dust_emis_fact dyn="mpas" phys="cam7" >2.30D0</dust_emis_fact> | |||
| <dust_emis_fact dyn="se" hgrid="ne30np4" phys="cam6" chem="trop_strat_mam4_vbs">0.8D0</dust_emis_fact> | |||
| <dust_emis_fact dyn="se" hgrid="ne30np4" phys="cam7" chem="trop_strat_mam4_vbs">0.8D0</dust_emis_fact> | |||
There was a problem hiding this comment.
Is this right for cam7 with Leung dust?
There was a problem hiding this comment.
@PeterHjortLauritzen @adamrher - I need you to answer this one also
There was a problem hiding this comment.
Good catch -- all cam7 dust_emis_fact's should be 2.3, so we should change this trop_strat_mam4_vbs entry to 2.3.
| @@ -2563,7 +2564,8 @@ | |||
| <dust_emis_fact hgrid="0.47x0.63" offline_dyn="1" phys="cam7" ver="chem">0.9D0</dust_emis_fact> | |||
|
|
|||
| <!-- dust emissions method --> | |||
| <dust_emis_method>Leung_2023</dust_emis_method> | |||
| <dust_emis_method> Zender_2003</dust_emis_method> | |||
| <dust_emis_method phys="cam7" > Leung_2023</dust_emis_method> | |||
|
@adamrher - could you please re-review and approve the latest dust_emis_fact change matches what we talked about offline and that you don't see any problems with these extensive changes |
|
@PeterHjortLauritzen - It looks like I should have you double check this as well - I asked for re-review |
Closes #1251
Closes #1249
No need for this test anymore. I checked out a CESM configuration and used this branch in it. I then ran a create_newcase using the same compset/grid that Cecile is using and confirmed that I got all of the requested settings after running preview_namelists. [@cecilehannay - Could you please checkout this branch and make sure that you get the namelist settings you want? I tested it with an FLTHIST and it worked.}
@briandobbins - Does this PR indeed close your issue?
@ekluzek - Including you in case you see any problems