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

Add a global ocean test case for computing and running with data (prescribed) melt rates #580

Merged
merged 5 commits into from
Jun 15, 2023

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Mar 29, 2023

This merge also adds a step for staging the remapped melt fluxes in the appropriate directory for use in E3SM. It can be used to compute data melt fluxes for existing meshes.

Checklist

  • User's Guide has been updated
  • Developer's Guide has been updated
  • API documentation in the Developer's Guide (api.rst) has any new or modified class, method and/or functions listed
  • Documentation has been built locally and changes look as expected
  • The E3SM-Project submodule has been updated with relevant E3SM changes
  • 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

@xylar xylar added enhancement New feature or request ocean in progress This PR is not ready for review or merging E3SM PR required labels Mar 29, 2023
@xylar xylar self-assigned this Mar 29, 2023
@xylar xylar force-pushed the add_prescribed_melt_fluxes branch from 572666a to a12e247 Compare March 29, 2023 13:21
@xylar
Copy link
Collaborator Author

xylar commented Mar 29, 2023

The corresponding E3SM PR is at E3SM-Ocean-Discussion/E3SM#45 for discussion.

@xylar xylar changed the title Add a global ocean test case for computing and running with prescribed melt rates Add a global ocean test case for computing and running with data (prescribed) melt rates Apr 13, 2023
@xylar xylar force-pushed the add_prescribed_melt_fluxes branch 2 times, most recently from 9fd0259 to 2929f71 Compare April 13, 2023 15:38
@xylar xylar force-pushed the add_prescribed_melt_fluxes branch from 2929f71 to 64d6db7 Compare April 29, 2023 11:42
@xylar xylar force-pushed the add_prescribed_melt_fluxes branch from 64d6db7 to 6f5b837 Compare May 15, 2023 21:21
@xylar
Copy link
Collaborator Author

xylar commented Jun 14, 2023

Needs to be rebased following #648, which brings in the required E3SM changes.

@xylar xylar force-pushed the add_prescribed_melt_fluxes branch from 6f5b837 to fd8376f Compare June 15, 2023 12:17
@xylar xylar added E3SM PR finished and removed in progress This PR is not ready for review or merging E3SM PR required labels Jun 15, 2023
@xylar
Copy link
Collaborator Author

xylar commented Jun 15, 2023

Testing

I ran the following test suites successfully on Chrysalis with Intel and OpenMPI:

@xylar xylar merged commit 19bae39 into MPAS-Dev:main Jun 15, 2023
@xylar xylar deleted the add_prescribed_melt_fluxes branch June 15, 2023 16:47
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.

1 participant