Testing

Here's what the baroclinic channel looks like with the tight axis limits:

Whereas before this PR, it would have looked like:

@cbegeman, are you able to review this? I think you had stronger opinions than I did on these particular plots.

Sure, I'll review.

@andrewdnolan Have you tested this for any case that are planar but non-periodic? As long as that looks like our "old" plot, then I'm happy with this change.

@cbegeman Good idea!

Here's the output for barotropic gyre (run at 10 km resolution).

This PR:

Head of main:

It's not the easiest to see the boundaries, but they seem just about the same to me. If you have idea of other tasks that are planar (non-periodic) but that can run at coarser resolution, please let me know.

If you have time to rebase onto #353, that would give you a test case with non-periodic boundaries at very coarse resolution

Testing (Cont.)

Hmm, seems like the barotropic gyre is periodic in the y-dimension. But we can still take a look at the plots:

This PR:

Prior to this PR:

Unrelated to this PR, but I'd like to confirm the vertex patches are being properly handled along the boundary here...

Hmm, seems like the barotropic gyre is periodic in the y-dimension.

Yes, sorry, should have mentioned that.

The non-periodic boundary looks good though! Thanks for taking the time to run this case.

Unrelated to this PR, but I'd like to confirm the vertex patches are being properly handled along the boundary here...

That would be fantastic.

@andrewdnolan, I set you as the assignee so feel free to merge when you're ready.