Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore the water thickness head in the channel model #823

Open
wants to merge 1 commit into
base: landice/develop
Choose a base branch
from

Conversation

matthewhoffman
Copy link
Member

For calculations of hydropotential gradient in the channel model, this
commit swtiches from using the full hydropotential to hydropotentialBase
(which ignores the hydropotential contribution from the thickness of the
water).

This seems to prevent channel blowups around lakes. It also might not
be modeling channel interactiosn with lakes properly. This requires
further investigation.

For calculations of hydropotential gradient in the channel model, this
commit swtiches from using the full hydropotential to hydropotentialBase
(which ignores the hydropotential contribution from the thickness of the
water).

This seems to prevent channel blowups around lakes.  It also might not
be modeling channel interactiosn with lakes properly.  This requires
further investigation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant