Skip to content

New ridge ratio code#494

Merged
einola merged 4 commits intodevelopfrom
issue442_new_ridge_ratio
Dec 16, 2020
Merged

New ridge ratio code#494
einola merged 4 commits intodevelopfrom
issue442_new_ridge_ratio

Conversation

@einola
Copy link
Copy Markdown
Member

@einola einola commented Dec 14, 2020

New ridge ratio code to address issue #442. What it does:

  • When ice forms (thermodynamically) we conserve the volume of ridged ice (i.e., ridges don't grow thermodynamically)

  • When ice melts we conserve the ridge ratio (i.e., ridges and level ice melt at the same rate).

  • When thick ice converges we conserve the mean thickness of level ice

  • When thin (young) ice converges we conserve the volume of level ice

…g). We have three cases: 1) Thermodynamics: Conserve volume of level ice. 2) Ridging of thick ice: Conserve mean thickness of level ice (or volume per ice covered area). 3) Ridging of thin ice: Conserve volume of level ice.
Copy link
Copy Markdown
Contributor

@docguibou docguibou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I get the idea and it seems all fine to me, but I am confused by this C_n in your comments @einola . What is it?

@einola
Copy link
Copy Markdown
Member Author

einola commented Dec 14, 2020

Ah, yes. C_n in the comments is concentration at time step n. Normally it's A, it's true, but I'd already used A for area ... o_o

Copy link
Copy Markdown
Contributor

@docguibou docguibou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I understand better, thanks. Double checked quickly and normally there is no need for more "capping check", as you said (as long as conc and thick are positive at least, but well...).

Comment thread model/finiteelement.cpp Outdated
As pointed out by Heather
@einola einola merged commit d60540c into develop Dec 16, 2020
@einola einola deleted the issue442_new_ridge_ratio branch December 16, 2020 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants