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 a step for smoothing topograpy #888

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Jan 24, 2025

We remap the topography once without smoothing and then once with smoothing. The unsmoothed topography is used to determine the land mask, thus allowing many meshes with different levels of smoothing to have the same mapping files in E3SM.

For the time being, only two simple types of smoothing are supported: a smoothing over a fixed distance or by expanding cells by a fixed fraction. (The two types of smoothing can also be used at the same time.)

In the future, maps with non-uniform smoothing could be added for finer control.

A check is used to bypass the second smoothing pass if, in fact, no smoothing is requested (the default behavior).

Checklist

  • User's Guide has been updated
  • Developer's Guide has been updated
  • API documentation in the Developer's Guide (api.rst) has any new or modified class, method and/or functions listed
  • Documentation has been built locally and changes look as expected
  • Document (in a comment titled Testing in this PR) any testing that was used to verify the changes

We remap the topography once without smoothing and then once
with smoothing.  The unsmoothed topography is used to determine
the land mask, thus allowing many meshes with different levels
of smoothing to have the same mapping files in E3SM.

For the time being, only two simple types of smoothing are supported:
a smoothing over a fixed distance or by expanding cells by a fixed
fraction.  (The two types of smoothing can also be used at the
same time.)

In the future, maps with non-uniform smoothing could be added for
finer control.

A check use used to bypass the second smoothing pass if, in fact,
no smoothing is requested (the default behavior).
@xylar xylar added ocean in progress This PR is not ready for review or merging labels Jan 24, 2025
@xylar xylar requested a review from bmooremaley January 24, 2025 16:22
@xylar xylar self-assigned this Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in progress This PR is not ready for review or merging ocean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant