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

Add support for maximum allowed sea-ice thickness #998

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Mar 30, 2024

This allows us to mask out "bad" sea-ice cells that accumulate unphysical amounts of sea ice from the area, volume and mean thickness time series.

This is needed in order for us to produce useful analysis for the SORRM v2.1 simulations, where there is a cell with 90 km of sea ice(!!!) that we need to mask out.

This allows us to mask out "bad" sea-ice cells that accumulate
unphysical amounts of sea ice from the area, volume and mean
thickness time series
@xylar
Copy link
Collaborator Author

xylar commented Mar 30, 2024

@darincomeau, we'll want to test this on the v2.1 cryo runs.

@darincomeau
Copy link
Contributor

Thanks @xylar ! I'll test this out with the v2.1 cryo runs.

@darincomeau
Copy link
Contributor

Here's years 700-100 of v2.1 SORRM 1950 control before this:
image

And with this, the following added to the config file

[timeSeriesSeaIceAreaVol]
maxAllowedSeaIceThickness = 20

image

Copy link
Contributor

@darincomeau darincomeau left a comment

Choose a reason for hiding this comment

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

Approved based on above testing. Thanks @xylar !

@darincomeau
Copy link
Contributor

also, saw no change in the Arctic sea ice volume time series.

@xylar
Copy link
Collaborator Author

xylar commented Apr 6, 2024

@milenaveneziani, wrong email, I think :-)

@milenaveneziani
Copy link
Collaborator

argh, sorry about that!

I do have a comment on this though. I suppose this is the problem that was mentioned at the polar meeting on Wed. I didn't realize this was so huge.. In the past, when we were running with the old ARRM mesh, we used to have big build ups of sea ice in a couple of Arctic grid points, but that was not even visible in the MPAS-Analysis diagnostics. This must be related to something recent, as I also don't remember it happening in previous SORRM simulations.
Are we sure we want to tell MPAS-Analysis to ignore these biases? I guess the good thing is that the default value of max Allowed sea ice thickness is 0.

@xylar
Copy link
Collaborator Author

xylar commented Apr 6, 2024

@milenaveneziani,

This is with the SORRM v2.1 mesh. We ran this 1000 year simulation and we need MPAS-Analysis to work with it even though there's a single cell with 90 km of sea ice. So, yes, we need this capability to produce useful analysis even if we certainly don't want this on by default.

We believe the problem (sea ice cells that have only once edge that is not either land or ice shelf) has been fixed in v3 meshes (see MPAS-Dev/compass#752) but that doesn't help with the v2.1 simulations

@milenaveneziani
Copy link
Collaborator

I see, thanks for clarifying Xylar.

@xylar xylar merged commit da72e59 into MPAS-Dev:develop Apr 12, 2024
5 checks passed
@xylar xylar deleted the mask-very-thick-ice branch April 12, 2024 21:05
@xylar xylar removed the in progress label Apr 12, 2024
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.

None yet

3 participants