Skip to content

github action for running a coarse gis meshgen test#899

Merged
xylar merged 4 commits intoMPAS-Dev:mainfrom
SCOREC:cws/gisCoarseMeshAction
Apr 3, 2025
Merged

github action for running a coarse gis meshgen test#899
xylar merged 4 commits intoMPAS-Dev:mainfrom
SCOREC:cws/gisCoarseMeshAction

Conversation

@cwsmith
Copy link
Copy Markdown
Contributor

@cwsmith cwsmith commented Apr 2, 2025

This PR adds a github action workflow for running a coarse GIS meshgen case on a 'free' github hosted runner. The run takes about 11 minutes. It is only triggered when a PR, in any of its commits, changes either of the following files:

https://github.com/SCOREC/compass/blob/368a4a5b0d22344b53c08dbd2c8fbebf6b738032/.github/workflows/gis_coarse_meshgen.yml#L7-L8

    - compass/landice/mesh.py
    - compass/landice/tests/greenland/mesh.py

Testing was done here by pushing changes to compass/landice/tests/mesh.py. Those changes were reverted, but the action log is available here: https://github.com/MPAS-Dev/compass/actions/runs/14233471393/job/39888454918

As noted below in the checklist, this test is currently downloading the GIS input data from a public RPI box directory. Is there a better place to host these files?

Checklist

  • Document any testing that was used to verify the changes
  • Find a better home for the GIS input data.

@cwsmith cwsmith force-pushed the cws/gisCoarseMeshAction branch from 16a412b to 368a4a5 Compare April 3, 2025 02:31
@cwsmith cwsmith marked this pull request as ready for review April 3, 2025 02:36
@xylar xylar self-assigned this Apr 3, 2025
@xylar xylar added enhancement New feature or request land ice ci Changes affect Azure Pipelines CI labels Apr 3, 2025
Copy link
Copy Markdown
Collaborator

@xylar xylar left a comment

Choose a reason for hiding this comment

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

Great, thanks @cwsmith! Let's give this a try!

@xylar xylar merged commit db69d96 into MPAS-Dev:main Apr 3, 2025
12 checks passed
@xylar
Copy link
Copy Markdown
Collaborator

xylar commented Apr 3, 2025

Find a better home for the GIS input data.

It could potentially be hosted on the LCRC server. We've tried to be careful not to overload the server but this wouldn't be big and wouldn't run all that often.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes affect Azure Pipelines CI enhancement New feature or request land ice

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants