Skip to content

Commit

Permalink
clearer documentation for correct_leap_year_inconsistency flag
Browse files Browse the repository at this point in the history
  • Loading branch information
MJHarrison-GFDL committed Sep 1, 2021
1 parent d37f1cf commit 88da4c9
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 15 deletions.
12 changes: 4 additions & 8 deletions config_src/infra/FMS1/MOM_interp_infra.F90
Original file line number Diff line number Diff line change
Expand Up @@ -247,14 +247,10 @@ integer function init_extern_field(file, fieldname, MOM_domain, domain, verbose,
!! fatal error if the axis Cartesian attribute is
!! not set to a recognized value.
logical, optional, intent(in) :: correct_leap_year_inconsistency !< If present and true,
!! Turns on a kluge for an inconsistency which may occur in a special case.
!! When the modulo time period (i.e. Time_end - Time_beg) is a whole number of years
!! and is not a multiple of 4, and the calendar in use has leap years, then it is
!! likely that the interpolation will involve mapping a common year onto a leap year.
!! In this case it is often desirable, but not absolutely necessary, to use data for
!! Feb 28 of the leap year when it is mapped onto a common year.
!! To turn this on, set correct_leap_year_inconsistency=.true.

!! then if, (1) a calendar containing leap years
!! is in use, and (2) the modulo time period of the
!! data is an integer number of years, then map
!! a model date of Feb 29. onto a common year on Feb. 28.

if (present(MOM_Domain)) then
init_extern_field = init_external_field(file, fieldname, domain=MOM_domain%mpp_domain, &
Expand Down
12 changes: 5 additions & 7 deletions config_src/infra/FMS2/MOM_interp_infra.F90
Original file line number Diff line number Diff line change
Expand Up @@ -247,13 +247,11 @@ integer function init_extern_field(file, fieldname, MOM_domain, domain, verbose,
!! fatal error if the axis Cartesian attribute is
!! not set to a recognized value.
logical, optional, intent(in) :: correct_leap_year_inconsistency !< If present and true,
!! Turns on a kluge for an inconsistency which may occur in a special case.
!! When the modulo time period (i.e. Time_end - Time_beg) is a whole number of years
!! and is not a multiple of 4, and the calendar in use has leap years, then it is
!! likely that the interpolation will involve mapping a common year onto a leap year.
!! In this case it is often desirable, but not absolutely necessary, to use data for
!! Feb 28 of the leap year when it is mapped onto a common year.
!! To turn this on, set correct_leap_year_inconsistency=.true.
!! then if, (1) a calendar containing leap years
!! is in use, and (2) the modulo time period of the
!! data is an integer number of years, then map
!! a model date of Feb 29. onto a common year on Feb. 28.



if (present(MOM_Domain)) then
Expand Down

0 comments on commit 88da4c9

Please sign in to comment.