Add roughness length over land to the perturbed variables #2
Conversation
There was a problem hiding this comment.
This looks pretty straightforward. We should think about conditionally allocating the arrays in FV3's stochastic_physics_wrapper and passing them as either optional arguments or with assumed size to lnd_apply_perts to reduce our memory footprint.
We will also need a ufs-weather-model PR that combines these changes and tests them. I can do that if you want me to.
compns_stochy.F90
Outdated
| @@ -294,7 +294,7 @@ subroutine compns_stochy (me,sz_nml,input_nml_file,fn_nml,nlunit,deltim,iret) | |||
| 'land perturbations will be applied to selected paramaters, using newer scheme designed for DA ens spread' | |||
| do k =1,n_var_lndp | |||
| select case (lndp_var_list(k)) | |||
| case('vgf','smc','stc','alb', 'sal','emi') | |||
| case('vgf','smc','stc','alb', 'sal','emi','zol','zoi','zow') | |||
There was a problem hiding this comment.
Should this be just zol for the time being? I see that the code will stop anyway in lndp_apply_perts, so I think either way is fine.
There was a problem hiding this comment.
Yes, it should. Sorry, forgot to remove them.
There was a problem hiding this comment.
@DomHeinzeller Yes, it could be a very good change to allocate arrays conditionally.
I'll appreciate if you could do the ufs-weather-model PR. Thank you.
There was a problem hiding this comment.
Can you please make the change in compns_stochy.F90 so that it reads and then push it to this PR?
case('vgf','smc','stc','alb','sal','emi','zol')
…rturbed land variables.
DomHeinzeller
changed the title
The zorll variable is added to the perturbed land parameters.