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 grow mask to constrain flood fill mask #560

Merged
merged 2 commits into from
Mar 26, 2024

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Mar 26, 2024

This merge adds a daGrow argument to compute_mpas_flood_fill_mask() that can be used to constrain where we are allowed to flood. Previously, this function was only applied on culled meshes and was used to make sure all cells in the culled mesh were connected to each other. With this change, the function can be used for more general flood fills.

This allows a flood-fill to fill in a masked region rather than
the entire (presumably culled) mesh.
@xylar xylar self-assigned this Mar 26, 2024
@xylar
Copy link
Collaborator Author

xylar commented Mar 26, 2024

Testing

The daGrow argument has been used in Compass to make sure the land-ice mask does not have holes (by flood filling an "ocean" mask that is the inverse of the land-ice mask) and then to ensure that all land-ice cells are connected to each other (by flood filling the land-ice mask from the south pole). Tests were performed on IcoswISC meshes at both 240 km and 30 km resolution.

@xylar xylar merged commit bd8aebf into MPAS-Dev:master Mar 26, 2024
7 checks passed
@xylar xylar deleted the add-grow-mask-to-flood-fill branch March 26, 2024 11:13
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.

1 participant