Perform flood fills on the land-ice mask

MPAS-Dev / compass

Configuration Of MPAS Setups

Other

10 stars 37 forks source link

Perform flood fills on the land-ice mask #800

Closed xylar closed 5 months ago

xylar commented 5 months ago

This merge makes 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 ensures that all land-ice cells are connected to each other (by flood filling the land-ice mask from the south pole).

Checklist

[x] Document (in a comment titled Testing in this PR) any testing that was used to verify the changes

xylar commented 5 months ago

Testing

I uses this branch on IcoswISC240, IcoswISC30 and RRSwISC6to18 meshes. In all cases, "islands" of isolated land-ice were removed from landIceMask. In the last of these cases, holes in landIceMask were also filled in, see https://github.com/MPAS-Dev/compass/pull/801#issuecomment-2021527308

xylar commented 5 months ago

I also ran the pr suite and it completed successfully.