Skip to content

Commit

Permalink
MOM6: (*)Fixes gprime(1) when no layer coordinates are set
Browse files Browse the repository at this point in the history
    - The "none" option for COORD_CONFIG literally did nothing which
      meant that Rlay and gprime were unset (they actually appeared to
      be zero).  This mode was added with the intent that the model
      should not need these coordinate variables in ALE mode. However,
      gprime(k=1) is used and should be set to GFS, which this commit
      now does.
    - I also added "ALE" as an option for COORD_CONFIG which is
      equivalent to "none" since "ALE" is bit more intuitive.
      - We might even use this as a master switch or check that
        COORD_CONFIG is consistent with USE_REGRIDDING (for later)?
    - This commit "appears" to changes answers but I don't think it
      does. What changes is the calculation of energy for some
      experiments in which Rlay was not set before this commit.
      A non-zero gprime(1) produces different APEs due to sea surface
      height calculations that are only accurate to within roundoff
      compared with the depth of the ocean.
      Experiments affected are:
        - CVmix_SCM_tests/cooling_only/EPBL
        - CVmix_SCM_tests/cooling_only/KPP
        - CVmix_SCM_tests/mech_only/EPBL
        - CVmix_SCM_tests/mech_only/KPP
        - CVmix_SCM_tests/skin_warming_wind/EPBL
        - CVmix_SCM_tests/skin_warming_wind/KPP
        - CVmix_SCM_tests/wind_only/EPBL
        - CVmix_SCM_tests/wind_only/KPP
        - SCM_idealized_hurricane
        - single_column/EPBL
        - single_column/KPP
        - unit_tests
      All the above have COORD_CONFIG="none".
MOM6 commit NOAA-GFDL/MOM6@4e8790b
  • Loading branch information
Hallberg-NOAA committed Jul 28, 2017
1 parent 829ddea commit f30621c
Show file tree
Hide file tree
Showing 119 changed files with 143 additions and 1 deletion.
1 change: 1 addition & 0 deletions coupled_AM2_LM3_SIS/CM2G63L/MOM_parameter_doc.all
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions coupled_AM2_LM3_SIS/CM2G63L/MOM_parameter_doc.short
Original file line number Diff line number Diff line change
Expand Up @@ -152,6 +152,7 @@ CHANNEL_CONFIG = "global_1deg" ! default = "none"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@ CHANNEL_CONFIG = "global_1deg" ! default = "none"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@ CHANNEL_CONFIG = "global_1deg" ! default = "none"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/Baltic/MOM_parameter_doc.all
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/Baltic/MOM_parameter_doc.short
Original file line number Diff line number Diff line change
Expand Up @@ -152,6 +152,7 @@ CHANNEL_CONFIG = "global_1deg" ! default = "none"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -410,6 +410,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -165,6 +165,7 @@ CHANNEL_CONFIG = "global_1deg" ! default = "none"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/OM4_025/MOM_parameter_doc.all
Original file line number Diff line number Diff line change
Expand Up @@ -395,6 +395,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/OM4_025/MOM_parameter_doc.short
Original file line number Diff line number Diff line change
Expand Up @@ -181,6 +181,7 @@ CHANNEL_LIST_FILE = "MOM_channels_global_025" ! default = "MOM_channel_list"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/OM4_05/MOM_parameter_doc.all
Original file line number Diff line number Diff line change
Expand Up @@ -395,6 +395,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/OM4_05/MOM_parameter_doc.short
Original file line number Diff line number Diff line change
Expand Up @@ -174,6 +174,7 @@ CHANNEL_LIST_FILE = "MOM_channels_global_025" ! default = "MOM_channel_list"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/SIS2/MOM_parameter_doc.all
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/SIS2/MOM_parameter_doc.short
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@ CHANNEL_CONFIG = "global_1deg" ! default = "none"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/SIS2_bergs_cgrid/MOM_parameter_doc.all
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/SIS2_bergs_cgrid/MOM_parameter_doc.short
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@ CHANNEL_CONFIG = "global_1deg" ! default = "none"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/SIS2_cgrid/MOM_parameter_doc.all
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
1 change: 1 addition & 0 deletions ice_ocean_SIS2/SIS2_cgrid/MOM_parameter_doc.short
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@ CHANNEL_CONFIG = "global_1deg" ! default = "none"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -383,6 +383,7 @@ OMEGA = 7.2921E-05 ! [s-1] default = 7.2921E-05
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -182,6 +182,7 @@ CHANNEL_LIST_FILE = "MOM_channels_FLOR" ! default = "MOM_channel_list"
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -359,6 +359,7 @@ BETA = 0.0 ! [m-1 s-1] default = 0.0
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ F_0 = 1.0E-04 ! [s-1] default = 0.0
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -361,6 +361,7 @@ BETA = 0.0 ! [m-1 s-1] default = 0.0
! === module MOM_coord_initialization ===
COORD_CONFIG = "none" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand All @@ -376,6 +377,8 @@ COORD_CONFIG = "none" !
! ts_profile - use temperature and salinity profiles
! (read from COORD_FILE) to set layer densities.
! USER - call a user modified routine.
GFS = 9.80616 ! [m s-2] default = 9.80616
! The reduced gravity at the free surface.
BOUNDARY_EXTRAPOLATION_PRESSURE = True ! [Boolean] default = True
! When defined, the reconstruction is extrapolated
! within boundary cells rather than assume PCM for the.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,7 @@ F_0 = 1.0E-04 ! [s-1] default = 0.0
! === module MOM_coord_initialization ===
COORD_CONFIG = "none" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -361,6 +361,7 @@ BETA = 0.0 ! [m-1 s-1] default = 0.0
! === module MOM_coord_initialization ===
COORD_CONFIG = "none" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand All @@ -376,6 +377,8 @@ COORD_CONFIG = "none" !
! ts_profile - use temperature and salinity profiles
! (read from COORD_FILE) to set layer densities.
! USER - call a user modified routine.
GFS = 9.80616 ! [m s-2] default = 9.80616
! The reduced gravity at the free surface.
BOUNDARY_EXTRAPOLATION_PRESSURE = True ! [Boolean] default = True
! When defined, the reconstruction is extrapolated
! within boundary cells rather than assume PCM for the.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,7 @@ F_0 = 1.0E-04 ! [s-1] default = 0.0
! === module MOM_coord_initialization ===
COORD_CONFIG = "none" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -359,6 +359,7 @@ BETA = 0.0 ! [m-1 s-1] default = 0.0
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ F_0 = 1.0E-04 ! [s-1] default = 0.0
! === module MOM_coord_initialization ===
COORD_CONFIG = "file" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -361,6 +361,7 @@ BETA = 0.0 ! [m-1 s-1] default = 0.0
! === module MOM_coord_initialization ===
COORD_CONFIG = "none" !
! This specifies how layers are to be defined:
! ALE or none - used to avoid defining layers in ALE mode
! file - read coordinate information from the file
! specified by (COORD_FILE).
! BFB - Custom coords for buoyancy-forced basin case
Expand All @@ -376,6 +377,8 @@ COORD_CONFIG = "none" !
! ts_profile - use temperature and salinity profiles
! (read from COORD_FILE) to set layer densities.
! USER - call a user modified routine.
GFS = 9.80616 ! [m s-2] default = 9.80616
! The reduced gravity at the free surface.
BOUNDARY_EXTRAPOLATION_PRESSURE = True ! [Boolean] default = True
! When defined, the reconstruction is extrapolated
! within boundary cells rather than assume PCM for the.
Expand Down
Loading

0 comments on commit f30621c

Please sign in to comment.