-
Notifications
You must be signed in to change notification settings - Fork 35
Description
Hi everyone i have some trouble with the Langmuir parameterisation in cvmix_kpp.F90...
CVMix-src/src/shared/cvmix_kpp.F90
Lines 644 to 657 in a4a6fb0
| call cvmix_coeffs_kpp(new_Mdiff, new_Tdiff, new_Sdiff, & | |
| CVmix_vars%zw_iface, CVmix_vars%zt_cntr, & | |
| CVmix_vars%Mdiff_iface, CVmix_vars%Tdiff_iface, & | |
| CVMix_vars%Sdiff_iface, & | |
| CVmix_vars%BoundaryLayerDepth, & | |
| CVmix_vars%kOBL_depth, & | |
| CVmix_vars%kpp_Tnonlocal_iface, & | |
| CVmix_vars%kpp_Snonlocal_iface, & | |
| CVmix_vars%SurfaceFriction, & | |
| CVmix_vars%SurfaceBuoyancyForcing, & | |
| nlev, max_nlev, & | |
| CVmix_vars%LangmuirEnhancementFactor, & | |
| CVmix_vars%StokesMostXi, & | |
| CVmix_kpp_params_user) |
... there the Langmuir Enhancment Factor is setted as in input parameter
CVMix-src/src/shared/cvmix_kpp.F90
Line 655 in a4a6fb0
| CVmix_vars%LangmuirEnhancementFactor, & |
... its given as an optional parameter to
CVMix-src/src/shared/cvmix_kpp.F90
Lines 676 to 680 in a4a6fb0
| subroutine cvmix_coeffs_kpp_low(Mdiff_out, Tdiff_out, Sdiff_out, zw, zt, & | |
| old_Mdiff, old_Tdiff, old_Sdiff, OBL_depth, & | |
| kOBL_depth, Tnonlocal, Snonlocal, surf_fric,& | |
| surf_buoy, nlev, max_nlev, Langmuir_EFactor,& | |
| StokesXI,CVmix_kpp_params_user) |
CVMix-src/src/shared/cvmix_kpp.F90
Line 703 in a4a6fb0
| real(cvmix_r8), intent(in), optional :: Langmuir_EFactor |
It looks to me that the variable CVmix_vars%LangmuirEnhancementFactor, never get filled up with a sensible value but is later used in the computation at multiple locations. There seems to be no line with cvmix_put('LangmuirEnhancementFactor', val,....)
CVMix-src/src/shared/cvmix_kpp.F90
Lines 994 to 999 in a4a6fb0
| if (CVMix_KPP_Params_in%Langmuir_Mixing_Opt & | |
| .eq. LANGMUIR_MIXING_LWF16) then | |
| ! enhance the turbulent velocity scale | |
| wm_OBL = wm_OBL * Langmuir_EFactor | |
| ws_OBL = ws_OBL * Langmuir_EFactor | |
| end if |
CVMix-src/src/shared/cvmix_kpp.F90
Lines 1202 to 1210 in a4a6fb0
| select case (CVMix_KPP_Params_in%Langmuir_Mixing_Opt) | |
| case (LANGMUIR_MIXING_LWF16) | |
| MixingCoefEnhancement = Langmuir_EFactor | |
| case (LANGMUIR_MIXING_RWHGK16) | |
| MixingCoefEnhancement = cvmix_one + ShapeNoMatchAtS/NMshapeMax * & | |
| (Langmuir_EFactor - cvmix_one) | |
| case default | |
| MixingCoefEnhancement = cvmix_one | |
| end select |
thus i have my doubts that the options Langmuir_mixing_str = 'LWF16' and 'RWHGK16' can work, only Langmuir_mixing_str='NONE' is possible. Can you verify this?!
Thanks for you help !
kind regards