-
Notifications
You must be signed in to change notification settings - Fork 37
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 additional ice_shelf_2d test cases #685
Conversation
TestingTests were run on chrysalis with intel, openmpi. I have not yet done a comparison of this branch against a compass baseline. |
@cbegeman, I'm trying to run all the |
Compass is just hanging on the last one, though there is clearly an error message from MPAS-Ocean. I've been seeing that pretty often lately and have no idea why it hangs rather than returning with an error, but that isn't the primary problem. All |
@xylar Hmm, I'll look into it. |
062e4d4
to
1321ce3
Compare
1321ce3
to
9715be6
Compare
@xylar All of the tests run now. I reduced the time step for all cases, reduced the ice shelf basal slope, and removed the inflection point in the ice shelf basal slope. I also set the threshold for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cbegeman, these changes are really great to have! Most of my comments are just aesthetic and kind of meant to nudge things in a Polaris direction. But feel free to ignore whichever you don't think are worth fixing. We can leave some or all of this for later.
Co-authored-by: Xylar Asay-Davis <[email protected]>
bb0cd4c
to
a5a8c6b
Compare
@xylar I just needed a chance to migrate your suggestions, fixup, and test. I think it's looking good now, minus the edge issues with the plot_horiz images. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
This PR adds new
ice_shelf_2d
test cases which are combinations of:2km
resolutionsingle_layer
vertical coordinatetidal_forcing
version of the default test caseThere are also changes made to the cfg options related to the cavity geometry which I think are more intuitive for users looking to modify the cavity geometry. In addition, the
isomip_plus
plotter class was imported to generate new plots.Checklist
api.rst
) has any new or modified class, method and/or functions listedTesting
in this PR) any testing that was used to verify the changes