Skip to content

Commit 2398f3a

Browse files
brandon.reichlbrandon.reichl
authored andcommitted
Fix some potential diagnostic logic issue in MOM_diabatic_driver.
1 parent c7a29a5 commit 2398f3a

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

src/parameterizations/vertical/MOM_diabatic_driver.F90

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3417,14 +3417,17 @@ subroutine diabatic_driver_init(Time, G, GV, US, param_file, useALEalgorithm, di
34173417
CS%id_Bdif_ddiff_salt = register_diag_field('ocean_model',"Bflx_dia_diff_ddiff_salt", diag%axesTi, &
34183418
Time, "Diffusive diapycnal buoyancy flux across interfaces due to double diffusion of salt", &
34193419
"m2 s-3", conversion=GV%H_to_m**2*US%s_to_T**3)
3420+
! The following logic might be streamlined, but the repetitive calls should not do any harm
34203421
if (CS%id_Bdif>0 .or. CS%id_Bdif_ePBL>0) then
34213422
CS%Use_Kd_Work_or_N2_diag = .true.
34223423
CS%do_bflx_salt = .true.
34233424
CS%do_bflx_temp = .true.
3424-
elseif (CS%id_Bdif_salt>0 .or. CS%id_Bdif_ddiff_salt>0) then
3425+
endif
3426+
if (CS%id_Bdif_salt>0 .or. CS%id_Bdif_ddiff_salt>0) then
34253427
CS%Use_Kd_Work_or_N2_diag = .true.
34263428
CS%do_bflx_salt = .true.
3427-
elseif (CS%id_Bdif_temp>0 .or. CS%id_Bdif_ddiff_temp>0) then
3429+
endif
3430+
if (CS%id_Bdif_temp>0 .or. CS%id_Bdif_ddiff_temp>0) then
34283431
CS%Use_Kd_Work_or_N2_diag = .true.
34293432
CS%do_bflx_temp = .true.
34303433
endif
@@ -3447,14 +3450,17 @@ subroutine diabatic_driver_init(Time, G, GV, US, param_file, useALEalgorithm, di
34473450
CS%id_Bdif_dz_ddiff_salt = register_diag_field('ocean_model',"Bflx_dia_diff_dz_ddiff_salt", diag%axesTl, &
34483451
Time, "Layer integrated diffusive diapycnal buoyancy flux due to double diffusion of salt.", &
34493452
"m3 s-3", conversion=GV%H_to_m**3*US%s_to_T**3)
3453+
! The following logic might be streamlined, but the repetitive calls should not do any harm
34503454
if (CS%id_Bdif_dz>0 .or. CS%id_Bdif_dz_ePBL>0) then
34513455
CS%Use_Kd_Work_or_N2_diag = .true.
34523456
CS%do_bflx_salt = .true.
34533457
CS%do_bflx_temp = .true.
3454-
elseif (CS%id_Bdif_salt_dz>0 .or. CS%id_Bdif_dz_ddiff_salt>0) then
3458+
endif
3459+
if (CS%id_Bdif_salt_dz>0 .or. CS%id_Bdif_dz_ddiff_salt>0) then
34553460
CS%Use_Kd_Work_or_N2_diag = .true.
34563461
CS%do_bflx_salt = .true.
3457-
elseif (CS%id_Bdif_temp_dz>0 .or. CS%id_Bdif_dz_ddiff_temp>0) then
3462+
endif
3463+
if (CS%id_Bdif_temp_dz>0 .or. CS%id_Bdif_dz_ddiff_temp>0) then
34583464
CS%Use_Kd_Work_or_N2_diag = .true.
34593465
CS%do_bflx_temp = .true.
34603466
endif

0 commit comments

Comments
 (0)