You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The time varying forcing and the tidal potential forcing have an explicit dependence on time t that is currently not modified during the intermediate stages of RK4 and this results in a deterioration of the temporal convergence rate to first order from the expected fourth order. The computation of the forcing in MPAS-O during the time stepping procedure is carried out only at the beginning of the time step and not at the intermediate time levels required by RK4.
In the above branch the forward mode is also modified so that the call for ocn_time_varying_forcing_get is done within the forward mode only for split-explicit and not for RK4, see mpas_ocn_forward_mode.F after cloning the above branch.
As part of this fix we also modified the routine mpas_advance_forcing_clock to allow for more decimals when the dt is given in seconds.
The text was updated successfully, but these errors were encountered:
@sbrus89 I've tested a branch of MPAS-Ocean that has the source terms needed to evaluate mom del2 and del4 (with eventual application to Omega). I'm finding that we have to get this PR in to evaluate those terms, otherwise the error due to the forcing dominates. Do you have the time to revive this PR? E3SM-Ocean-Discussion#55
The time varying forcing and the tidal potential forcing have an explicit dependence on time
t
that is currently not modified during the intermediate stages ofRK4
and this results in a deterioration of the temporal convergence rate to first order from the expected fourth order. The computation of the forcing in MPAS-O during the time stepping procedure is carried out only at the beginning of the time step and not at the intermediate time levels required by RK4.I fixed this issue with @jeremy-lilly using a
forcingTimeIncrement
that takes into account the necessary additional contributions to the time levelt
, the changes are in this branch: https://github.com/gcapodag/MPAS-Model/commits/split_testingGrep on
forcingTimeIncrement
once cloned and see this commit if in need of additional details:gcapodag/MPAS-Model@9658985#diff-86c132deabb5d64780bda599d5222d5b67468c5d0c18bee8cad90f8fc1e4e3f7
In the above branch the forward mode is also modified so that the call for
ocn_time_varying_forcing_get
is done within the forward mode only forsplit-explicit
and not forRK4
, seempas_ocn_forward_mode.F
after cloning the above branch.As part of this fix we also modified the routine
mpas_advance_forcing_clock
to allow for more decimals when thedt
is given in seconds.The text was updated successfully, but these errors were encountered: