Skip to content

Add GDEX download tools to new utils/data_download folder - #981

Merged
xylar merged 2 commits into
MPAS-Dev:mainfrom
bmooremaley:gdex-download-tools
Sep 8, 2026
Merged

Add GDEX download tools to new utils/data_download folder#981
xylar merged 2 commits into
MPAS-Dev:mainfrom
bmooremaley:gdex-download-tools

Conversation

@bmooremaley

@bmooremaley bmooremaley commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

This PR adds GDEX download tools to a new data_download folder in the utils space. The download workflow builds requests to the GDEX API. There is an existing Python API client, but the tools in this PR use the Python standard library. This PR includes definitions for the NCEP Climate Forecast System Reanalysis (CFSR) and the Climate Forecast System Version 2 (CFSv2), but additional data sets available via GDEX can be added.

No changes have been made to the Compass documentation. Instead we have included a README.md to demonstrate basic usage.

Checklist

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

@bmooremaley

Copy link
Copy Markdown
Contributor Author

Testing

We have used this code to download the precipitation, pressure_surf, runoff and wind parameter groups in gdex_codes.yaml from CFSv2 for the period of 2011-01-01 through 2014-01-01 on Perlmutter.

@xylar

xylar commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

@bmooremaley, if you can take care of the linting issues (or specifically tell the linting tool to skip this util), I'd be fine to merge.

@xylar xylar self-assigned this Sep 8, 2026
@xylar xylar added the utility Utility script(s) outside of the compass package label Sep 8, 2026
@xylar
xylar self-requested a review September 8, 2026 19:55
@bmooremaley

Copy link
Copy Markdown
Contributor Author

@bmooremaley, if you can take care of the linting issues (or specifically tell the linting tool to skip this util), I'd be fine to merge.

Sure, handling that now...

@xylar xylar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good! Thanks for dealing with the linter.

@xylar
xylar enabled auto-merge September 8, 2026 20:25
@xylar
xylar merged commit b60c7fb into MPAS-Dev:main Sep 8, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

utility Utility script(s) outside of the compass package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants