Skip to content

Commit 58f43f4

Browse files
Raphael DussinRaphael Dussin
authored andcommitted
fix units and description (last fixes from review)
1 parent aab5f48 commit 58f43f4

File tree

2 files changed

+17
-13
lines changed

2 files changed

+17
-13
lines changed

src/parameterizations/lateral/MOM_internal_tides.F90

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -327,9 +327,13 @@ subroutine propagate_int_tide(h, tv, Nb, Rho_bot, dt, G, GV, US, inttide_input_C
327327
real :: En_initial, Delta_E_check ! Energies for debugging [H Z2 T-2 ~> m3 s-2 or J m-2]
328328
real :: TKE_Froude_loss_check, TKE_Froude_loss_tot ! Energy losses for debugging [H Z2 T-3 ~> m3 s-3 or W m-2]
329329
real :: HZ2_T3_to_W_m2 ! unit conversion factor for TKE from internal to mks
330+
! [H Z2 T-3 ~> m3 s-3 or W m-2]
330331
real :: HZ2_T2_to_J_m2 ! unit conversion factor for Energy from internal to mks
332+
! [H Z2 T-2 ~> m3 s-2 or J m-2]
331333
real :: W_m2_to_HZ2_T3 ! unit conversion factor for TKE from mks to internal
334+
! [m3 s-3 or W m-2 ~> H Z2 T-3]
332335
real :: J_m2_to_HZ2_T2 ! unit conversion factor for Energy from mks to internal
336+
! [m3 s-2 or J m-2 ~> H Z2 T-2]
333337
character(len=160) :: mesg ! The text of an error message
334338
integer :: En_halo_ij_stencil ! The halo size needed for energy advection
335339
integer :: a, m, fr, i, j, k, is, ie, js, je, isd, ied, jsd, jed, nAngle
@@ -425,8 +429,8 @@ subroutine propagate_int_tide(h, tv, Nb, Rho_bot, dt, G, GV, US, inttide_input_C
425429
call hchksum(CS%u_struct_bot(:,:,1), "Ustruct_bot mode 1", G%HI, haloshift=0, scale=US%m_to_Z)
426430
call hchksum(CS%u_struct_max(:,:,1), "Ustruct_max mode 1", G%HI, haloshift=0, scale=US%m_to_Z)
427431
call hchksum(CS%int_w2(:,:,1), "int_w2", G%HI, haloshift=0, scale=GV%H_to_MKS)
428-
call hchksum(CS%int_U2(:,:,1), "int_U2", G%HI, haloshift=0, scale=GV%H_to_m*US%m_to_Z**2)
429-
call hchksum(CS%int_N2w2(:,:,1), "int_N2w2", G%HI, haloshift=0, scale=GV%H_to_m*US%s_to_T**2)
432+
call hchksum(CS%int_U2(:,:,1), "int_U2", G%HI, haloshift=0, scale=GV%H_to_mks*US%m_to_Z**2)
433+
call hchksum(CS%int_N2w2(:,:,1), "int_N2w2", G%HI, haloshift=0, scale=GV%H_to_mks*US%s_to_T**2)
430434
endif
431435

432436
! Set the wave speeds for the modes, using cg(n) ~ cg(1)/n.**********************
@@ -1277,8 +1281,8 @@ subroutine itidal_lowmode_loss(G, GV, US, CS, Nb, Rho_bot, Ub, En, TKE_loss_fixe
12771281
real :: En_negl ! negligibly small number to prevent division by zero [H Z2 T-2 ~> m3 s-2 or J m-2]
12781282
real :: En_a, En_b ! energy before and after timestep [H Z2 T-2 ~> m3 s-2 or J m-2]
12791283
real :: I_dt ! The inverse of the timestep [T-1 ~> s-1]
1280-
real :: J_m2_to_HZ2_T2 ! unit conversion factor for Energy from mks to internal
1281-
real :: HZ2_T3_to_W_m2 ! unit conversion factor for Energy from internal to mks
1284+
real :: J_m2_to_HZ2_T2 ! unit conversion factor for Energy from mks to internal [m3 s-2 or J m-2 ~> H Z2 T-2]
1285+
real :: HZ2_T3_to_W_m2 ! unit conversion factor for Energy from internal to mks [H Z2 T-3 ~> m3 s-3 or W m-2]
12821286

12831287
is = G%isc ; ie = G%iec ; js = G%jsc ; je = G%jec
12841288

@@ -3244,7 +3248,7 @@ subroutine register_int_tide_restarts(G, GV, US, param_file, CS, restart_CS)
32443248

32453249
type(axis_info) :: axes_inttides(2)
32463250
real, dimension(:), allocatable :: angles, freqs ! Lables for angles and frequencies [nondim]
3247-
real :: HZ2_T2_to_J_m2 ! unit conversion factor for Energy from internal to mks
3251+
real :: HZ2_T2_to_J_m2 ! unit conversion factor for Energy from internal to mks [H Z2 T-2 ~> m3 s-2 or J m-2]
32483252

32493253
isd = G%isd ; ied = G%ied ; jsd = G%jsd ; jed = G%jed
32503254

@@ -3374,10 +3378,10 @@ subroutine internal_tides_init(Time, G, GV, US, param_file, diag, CS)
33743378
! nominal ocean depth, or a negative value for no limit [nondim]
33753379
real :: period_1 ! The period of the gravest modeled mode [T ~> s]
33763380
real :: period ! A tidal period read from namelist [T ~> s]
3377-
real :: HZ2_T2_to_J_m2 ! unit conversion factor for Energy from internal to mks
3378-
real :: HZ2_T3_to_W_m2 ! unit conversion factor for TKE from internal to mks
3379-
real :: W_m2_to_HZ2_T3 ! unit conversion factor for TKE from mks to internal
3380-
real :: J_m2_to_HZ2_T2 ! unit conversion factor for Energy from mks to internal
3381+
real :: HZ2_T2_to_J_m2 ! unit conversion factor for Energy from internal to mks [H Z2 T-2 ~> m3 s-2 or J m-2]
3382+
real :: HZ2_T3_to_W_m2 ! unit conversion factor for TKE from internal to mks [H Z2 T-3 ~> m3 s-3 or W m-2]
3383+
real :: W_m2_to_HZ2_T3 ! unit conversion factor for TKE from mks to internal [m3 s-3 or W m-2 ~> H Z2 T-3]
3384+
real :: J_m2_to_HZ2_T2 ! unit conversion factor for Energy from mks to internal [m3 s-2 or J m-2 ~> H Z2 T-2]
33813385
integer :: num_angle, num_freq, num_mode, m, fr
33823386
integer :: isd, ied, jsd, jed, a, id_ang, i, j, nz
33833387
type(axes_grp) :: axes_ang

src/parameterizations/vertical/MOM_internal_tide_input.F90

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -116,8 +116,8 @@ subroutine set_int_tide_input(u, v, h, tv, fluxes, itide, dt, G, GV, US, CS)
116116
! equation of state.
117117
logical :: avg_enabled ! for testing internal tides (BDM)
118118
type(time_type) :: time_end !< For use in testing internal tides (BDM)
119-
real :: HZ2_T3_to_W_m2 ! unit conversion factor for TKE from internal to mks
120-
real :: W_m2_to_HZ2_T3 ! unit conversion factor for TKE from mks to internal
119+
real :: HZ2_T3_to_W_m2 ! unit conversion factor for TKE from internal to mks [H Z2 T-3 ~> m3 s-3 or W m-2]
120+
real :: W_m2_to_HZ2_T3 ! unit conversion factor for TKE from mks to internal [m3 s-3 or W m-2 ~> H Z2 T-3]
121121

122122
integer :: i, j, is, ie, js, je, nz, isd, ied, jsd, jed
123123
integer :: i_global, j_global
@@ -404,8 +404,8 @@ subroutine int_tide_input_init(Time, G, GV, US, param_file, diag, CS, itide)
404404
real :: kappa_h2_factor ! factor for the product of wavenumber * rms sgs height [nondim].
405405
real :: kappa_itides ! topographic wavenumber and non-dimensional scaling [L-1 ~> m-1]
406406
real :: min_zbot_itides ! Minimum ocean depth for internal tide conversion [Z ~> m].
407-
real :: HZ2_T3_to_W_m2 ! unit conversion factor for TKE from internal to mks
408-
real :: W_m2_to_HZ2_T3 ! unit conversion factor for TKE from mks to internal
407+
real :: HZ2_T3_to_W_m2 ! unit conversion factor for TKE from internal to mks [H Z2 T-3 ~> m3 s-3 or W m-2]
408+
real :: W_m2_to_HZ2_T3 ! unit conversion factor for TKE from mks to internal [m3 s-3 or W m-2 ~> H Z2 T-3]
409409
integer :: tlen_days !< Time interval from start for adding wave source
410410
!! for testing internal tides (BDM)
411411
integer :: i, j, is, ie, js, je, isd, ied, jsd, jed

0 commit comments

Comments
 (0)