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

LandIce: Added tests using the FO depth integrated model #634

Merged
merged 4 commits into from
Aug 17, 2023

Conversation

mperego
Copy link
Contributor

@mperego mperego commented May 21, 2023

In this PR we add tests using the FO depth integrated model introduced in Albany PR sandialabs/Albany#956.

In particular we added decomposition and restart tests for Thwaites and Humboldt glaciers.

We also updated tests' input files to comply with Albany PR sandialabs/Albany#953, where options for integration rules changed.
Removed the unused and outdated input file albany_schoof_input.yaml

Testing

  • Checked that this PR does not modify results of existing tests.

Checklist

  • User's Guide has been updated
  • Document (in a comment titled Testing in this PR) any testing that was used to verify the changes
  • New tests have been added to a test suite

@mperego mperego requested a review from matthewhoffman May 21, 2023 01:41
@mperego mperego self-assigned this May 21, 2023
@matthewhoffman
Copy link
Member

Testing

I ran the full_integration suite with this branch merged into #641 (required to get the version of Albany that includes MOLHO). I compared the diagnostic velocity solution for the Thwaites decomp test between FO and MOLHO:

image

@matthewhoffman matthewhoffman mentioned this pull request Aug 4, 2023
64 tasks
@matthewhoffman matthewhoffman force-pushed the depth_integrated_model branch 5 times, most recently from 4d29c18 to fa5186e Compare August 4, 2023 19:02
Copy link
Member

@matthewhoffman matthewhoffman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mperego , thank you for adding these tests for the depth-integrated solver. I will rebase and merge this once #641 is merged.

Options for Cubature Rules changed in Albany,
consequently we are updating the tests input files.
Also removed the unused outdated input file albany_schoof_input.yaml
@xylar
Copy link
Collaborator

xylar commented Aug 4, 2023

#641 is merged.

mperego and others added 3 commits August 4, 2023 14:28
Added the following tests using the FO depth integrated model
introduced in Albany PR sandialabs/Albany#956:

landice/thwaites/fo-depthInt_decomposition_test
landice/thwaites/fo-depthInt_restart_test
landice/humboldt/mesh-3km_restart_test/velo-fo-depthInt_calving-von_mises_stress_damage-threshold_faceMelting
@matthewhoffman matthewhoffman force-pushed the depth_integrated_model branch from fa5186e to 0f6f234 Compare August 4, 2023 20:28
@matthewhoffman matthewhoffman self-requested a review August 4, 2023 20:29
Copy link
Member

@matthewhoffman matthewhoffman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re-review after rebase (required)

@matthewhoffman matthewhoffman merged commit 8caa397 into MPAS-Dev:main Aug 17, 2023
@matthewhoffman
Copy link
Member

(forgot to actually merge this after checks completed after rebase two weeks ago...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants