Skip to content

Commit

Permalink
Merge f115dfe into 8b9990e
Browse files Browse the repository at this point in the history
  • Loading branch information
pat-schmitt committed May 23, 2023
2 parents 8b9990e + f115dfe commit 62c7e24
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 5 deletions.
23 changes: 21 additions & 2 deletions docs/ice-dynamics.rst
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,26 @@ Numerics
Semi-implicit model (new default in v1.6)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

TODO: Patrick
This model solves a diffusion equation for the ice thickness :math:`h`,
assuming a trapezoidal bed (Equation 10 in [Oerlemans_1997]_). Numerically
the equation is implemented using the present time-step diffusivity :math:`D`
but the future time-step surface slope :math:`\partial (b + h) / \partial x`
(with the glacier bed :math:`b`), hence the name semi-implicit.

The solution of this equation involves a matrix inversion, which makes a
single solving step around 1.5 times slower compared to the "Flux-based"
model. However, the scheme is more stable hence each single solving step can
use a three-time larger :math:`\Delta t`. Overall this halves the total
runtime.

With the increased stability, we did not see numerical instabilities like
for the "Flux-based" model (see :ref:`pitfalls.numerics`). But as the
underlying equation is the same, we see the same non-mass-conserving
behaviour in very steep slopes [Jarosch_etal_2013]_.

In summary, the Semi-implicit model is faster and more robust than the
"Flux-based" model, but less flexible (currently only working with a single
trapezoidal flowline).

"Flux-based" model
~~~~~~~~~~~~~~~~~~
Expand Down Expand Up @@ -195,7 +214,7 @@ numerical equations on a forward staggered grid and we use an adaptive time
stepping scheme.

See :ref:`pitfalls.numerics` for an ongoing discussion about the limitations
of OGGM's numerical scheme!
of OGGM's "Flux-based" scheme!


MUSCLSuperBeeModel
Expand Down
10 changes: 7 additions & 3 deletions docs/pitfalls.rst
Original file line number Diff line number Diff line change
Expand Up @@ -89,11 +89,15 @@ RGI region level, so that you don't have to worry about it.

.. _pitfalls.numerics:

The numerical model in OGGM is numerically unstable in some conditions
======================================================================
The "Flux-based" model in OGGM is numerically unstable in some conditions
=========================================================================

**New in OGGM 1.6:** in OGGM 1.6 and above, we use a considerably improved
semi-implicit scheme. However, for technical reasons, the "flux-based"
scheme is still used for multiple flowlines.

OGGM uses a CFL criterion to decide on the timestep to use during the
ice dynamics model iteration. The numerical scheme of OGGM is fast and
ice dynamics model iteration. The "flux-based" scheme of OGGM is fast and
flexible (e.g. it allows to compute the ice flow on multiple flowlines),
but it is not following textbook recommendations on numerical stability.

Expand Down

0 comments on commit 62c7e24

Please sign in to comment.