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

support running landice/greenland/mesh_gen test case on workstations #890

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cwsmith
Copy link
Contributor

@cwsmith cwsmith commented Jan 31, 2025

This PR passes the parallel_executable to a few functions that were hardcoded to use srun. With the right input files, this allows testing on workstations with limited memory and cores.

Testing was done with subsampled GIS inputs (to 4km, thanks to @matthewhoffman) for the landice/greenland/mesh_gen test.

Checklist

  • User's Guide has been updated - no change needed
  • Developer's Guide has been updated - no change needed
  • API documentation in the Developer's Guide (api.rst) has any new or modified class, method and/or functions listed - no changes needed - the doc string for interp_gridded2mali was updated in compass/landice/mesh.py
  • Documentation has been built locally and changes look as expected - the new parallel_executable arg is rendered in the html
  • not applicable The E3SM-Project submodule has been updated with relevant E3SM changes
  • not applicable The MALI-Dev submodule has been updated with relevant MALI changes
  • Document (in a comment titled Testing in this PR) any testing that was used to verify the changes
  • not applicable? New tests have been added to a test suite

@cwsmith cwsmith marked this pull request as ready for review January 31, 2025 15:02
@matthewhoffman matthewhoffman self-requested a review January 31, 2025 16:41
@matthewhoffman matthewhoffman self-assigned this Jan 31, 2025
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