Skip to content

Conversation

@xylar
Copy link
Collaborator

@xylar xylar commented Feb 3, 2020

This feature performs an iterative weighted average of a field with a given smoothing weight and for a given number of iterations. Currently, it is only used for Cryosphere test cases (with ice-shelf cavities, wISC).

Also renames config option ...fill_bathymetry_holes from ...smooth_topography because this is a more accurate description of what is done when this flag is .true.

closes #436

... instead of "smooth_topography" because this is a more accurate
description of what is done when this flag is .true.
@xylar
Copy link
Collaborator Author

xylar commented Feb 3, 2020

Currently, I'm having trouble with scratch fields (at least on my laptop). I'll try on LANL IC tomorrow and ask @mark-petersen for help if I'm still not able to figure it out.

xylar added 2 commits February 3, 2020 10:40
This feature performs an iterative weighted average of a field
(currently bottomDepthObserved) with a given smoothing weight and
for a given number of iterations.  Currently, it is only used for
wISC setups.
@xylar xylar force-pushed the ocean/add_topo_smoothing branch from 9accc29 to 3deec68 Compare February 3, 2020 17:41
Previously, invalid cells were getting averaged in.
@xylar
Copy link
Collaborator Author

xylar commented Feb 3, 2020

Results EC60to30wISC

Below are the bathymetry, ice draft and fraction of ice-shelf cover, without smoothing followed by with smoothing with a weight (of the central cell relative to the average of its neighbors) of 0.9 and 6 iterations.

bot_orig
bot_smooth
draft_orig
draft_smooth
frac_orig
frac_smooth

@xylar
Copy link
Collaborator Author

xylar commented Feb 3, 2020

weight of 0.9 and 6 iterations might be too much for this data set, particularly because it was already smoothed at 0.1 degrees on the original grid, but it does demonstrate the concept. I think this amount of smoothing might be more appropriate at higher res and in particular for the BedMachine bathymetry (unsmoothed) that we will hopefully move to for Cryosphere configurations.

@xylar
Copy link
Collaborator Author

xylar commented Feb 3, 2020

Results SO60to10wISC

Same as above but at higher resolution.

bot_orig
bot_smooth
draft_orig
draft_smooth
frac_orig
frac_smooth

In my testing, the SSH adjustment phase failed, so there's clearly some work still to do to make that more robust but I will do that in #433 with the new bathymetry.

@xylar
Copy link
Collaborator Author

xylar commented Mar 11, 2020

@mark-petersen, I assume you can handle the registry conflict but let me know.

Copy link
Contributor

@mark-petersen mark-petersen left a comment

Choose a reason for hiding this comment

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

Looks good by visual inspection. Passes nightly regression suite on grizzly, gnu/optimized, intel/debug. I did not look at the actual smoothed bathymetry results, but @xylar's plots above are sufficient.

@mark-petersen mark-petersen merged commit 2823470 into MPAS-Dev:ocean/develop Mar 11, 2020
@xylar xylar deleted the ocean/add_topo_smoothing branch March 20, 2020 14:00
@xylar xylar mentioned this pull request May 7, 2020
ashwathsv pushed a commit to ashwathsv/MPAS-Model that referenced this pull request Jul 21, 2020
…elop

This feature performs an iterative weighted average of a field with a
given smoothing weight and for a given number of iterations. Currently,
it is only used for Cryosphere test cases (with ice-shelf cavities,
wISC).

Also renames config option ...fill_bathymetry_holes from
...smooth_topography because this is a more accurate description of what
is done when this flag is .true.

closes MPAS-Dev#436
mark-petersen added a commit to mark-petersen/MPAS-Model that referenced this pull request Jan 11, 2021
…elop

This feature performs an iterative weighted average of a field with a
given smoothing weight and for a given number of iterations. Currently,
it is only used for Cryosphere test cases (with ice-shelf cavities,
wISC).

Also renames config option ...fill_bathymetry_holes from
...smooth_topography because this is a more accurate description of what
is done when this flag is .true.

closes MPAS-Dev#436
caozd999 pushed a commit to caozd999/MPAS-Model that referenced this pull request Jan 14, 2021
…elop

This feature performs an iterative weighted average of a field with a
given smoothing weight and for a given number of iterations. Currently,
it is only used for Cryosphere test cases (with ice-shelf cavities,
wISC).

Also renames config option ...fill_bathymetry_holes from
...smooth_topography because this is a more accurate description of what
is done when this flag is .true.

closes MPAS-Dev#436
xylar added a commit to xylar/compass that referenced this pull request Feb 3, 2022
Following the SORRM mesh from compass/legacy, we perform 10
iterations of smoothing with a weight of 0.92.

For more info on the smoothing approach, see:
MPAS-Dev/MPAS-Model#440
For more on the chosen values (related to ice-shelf cavities), see:
MPAS-Dev/MPAS-Model#532 (comment)
xylar added a commit to xylar/compass that referenced this pull request Mar 20, 2022
Following the SORRM mesh from compass/legacy, we perform 10
iterations of smoothing with a weight of 0.92.

For more info on the smoothing approach, see:
MPAS-Dev/MPAS-Model#440
For more on the chosen values (related to ice-shelf cavities), see:
MPAS-Dev/MPAS-Model#532 (comment)
xylar added a commit to xylar/compass that referenced this pull request Mar 22, 2022
We perform 6 iterations of smoothing with a weight of 0.9.

For more info on the smoothing approach, see:
MPAS-Dev/MPAS-Model#440
For more on previously chosen values (related to ice-shelf
cavities), see:
MPAS-Dev/MPAS-Model#532 (comment)
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.

3 participants