Isca icon indicating copy to clipboard operation
Isca copied to clipboard

New namelist parameters for Held-Suarez forcing and spectral dynamics

Open gregcolyer opened this issue 6 years ago • 13 comments

Adds P00 (reference pressure used in Held-Suarez, not necessarily the same as surface pressure) to the namelist, and a new parameter lapse_rate for [adiabatic] lapse rate normalized to dry. lapse_rate is used only for classic newtonian_damping (original and exoplanet variants); there is also a similar but dimensional parameter lapse used only for top_down_newtonian_damping; the two could be consolidated (not done).

Also adds a new parameter reference_temp to the spectral dynamics namelist, previously hard-coded to 300 K.

gregcolyer avatar Dec 14 '17 10:12 gregcolyer

Looks fine to me, but happy to defer this to @Alexander-P and @jamesp as I don't ever use the Held-Suarez bit of the code.

sit23 avatar Dec 14 '17 11:12 sit23

Looks good. We should probably get rid of the alpha parameter that I added (and haven't really used) as this achieves the same thing

jamesp avatar Dec 14 '17 12:12 jamesp

Ah yes, I didn't notice alpha. Same as (KAPPA*lapse_rate)?

gregcolyer avatar Dec 14 '17 12:12 gregcolyer

Yep. In fact, let's just scrap the EXOPLANET2 conditional branch altogether

jamesp avatar Dec 14 '17 12:12 jamesp

I didn't notice that there was an existing normalized lapse rate thing, I may look into using that for top_down. This looks good to me

alex-r-p avatar Dec 14 '17 12:12 alex-r-p

There wasn't until now

gregcolyer avatar Dec 14 '17 12:12 gregcolyer

thanks @gregcolyer, I was just doing this!

jamesp avatar Dec 14 '17 12:12 jamesp

good reading comprehension, me!

alex-r-p avatar Dec 14 '17 12:12 alex-r-p

I think the default lapse (in K/km?) corresponds to a lapse_rate of 2/3, whereas 1 is needed for standard Held-Suarez. We could code different defaults for classic and top_down though, if you want to keep top_down the same.

gregcolyer avatar Dec 14 '17 13:12 gregcolyer

Following discussion with @Alexander-P: no need for different defaults, so I will code up this consolidation (remove lapse from namelist but set based on lapse_rate) and push here

gregcolyer avatar Dec 15 '17 14:12 gregcolyer

Side note: P00 defaults to 1E5 exactly, as in the Held-Suarez paper; this does not have to equal the reference surface pressure, but for consistency one should presumably set the latter to 1E5 (i.e. override the more precise default value) in the typical case. It's probably not appropriate to change either default (one should match the paper, the other should be the best available value for Earth), but perhaps the code should warn if they differ only by this order (i.e. a few per cent)?

gregcolyer avatar Dec 15 '17 17:12 gregcolyer

Shall we put this in, @gregcolyer?

jamesp avatar Sep 04 '18 15:09 jamesp

Testing now

gregcolyer avatar Sep 10 '18 11:09 gregcolyer