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

Update albany_input.yaml with new basal pressure setting #542

Merged

Conversation

trhille
Copy link
Collaborator

@trhille trhille commented Feb 22, 2023

Need to add Use Pressurized Bed Above Sea Level: False setting to albany_input.yaml for landice_humboldt_mesh-3km_restart_test_velo-fo_calving-von_mises_stress_damage-threshold_faceMelting test case to run, after sandialabs/Albany#863

Checklist

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

Need to add `Use Pressurized Bed Above Sea Level: False` setting
to albany_input.yaml for landice_humboldt_mesh-3km_restart_test_velo-fo_calving-von_mises_stress_damage-threshold_faceMelting
test case to run, after sandialabs/Albany#863
@trhille
Copy link
Collaborator Author

trhille commented Feb 22, 2023

Testing

Testing on Chicoma, built MALI at commit c19ffb1e9e262c0b2a2675a65999c438717ae7d6 (using mpas-framework/Makefile from E3SM-Project/master so I could build with gnu-cray target) with load_dev_compass_1.2.0-alpha.4_chicoma-cpu_gnu_mpich_albany.sh. All tests in full_integration passed except for landice_humboldt_mesh-3km_restart_test_velo-fo_calving-von_mises_stress_damage-threshold_faceMelting, which died with an error in log.albany.0000.out:

terminate called after throwing an instance of 'Teuchos::Exceptions::InvalidParameterName'
  what():  Error!  The parameter "Use Pressurized Bed Above Sea Level" does not exist
in the parameter (sub)list "Albany Parameters->Problem->LandIce BCs->BC 0->Basal Friction Coefficient".

When I added this commit, set up full_integration again, and re-ran, all tests passed execution and validation.

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.

@trhille , thanks for taking care of this!

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.

2 participants