Skip to content

add era5 lake cover an depth artifacts#154

Merged
AlexisRenchon merged 1 commit intomainfrom
ar/era5_lake_cover_and_depth
Apr 1, 2026
Merged

add era5 lake cover an depth artifacts#154
AlexisRenchon merged 1 commit intomainfrom
ar/era5_lake_cover_and_depth

Conversation

@AlexisRenchon
Copy link
Copy Markdown
Member

@AlexisRenchon AlexisRenchon commented Mar 31, 2026

Checklist:

  • I created a new folder $artifact_name
    • I added a README.md in that that folder that
      • describes the data and processing done to it
      • lists the sources of the raw data
      • lists the required citation, licenses
    • If applicable (e.g., for Creative Commons), I added a LICENSE file
    • I added the scripts that retrieve, process, and produce the artifact
    • I added the environment used for such scripts (typically, Project.toml
      and Manifest.toml)
    • I added the OutputArtifacts.toml file containing the information
      needed for package developers to add $artifact_name to their package
  • I uploaded the artifact folder to the Caltech cluster (in
    /resnick/groups/esm/ClimaArtifacts/artifacts/$artifact_name)
  • I added the relevant code to the Overides.toml on the Caltech Cluster
    (in /resnick/groups/esm/ClimaArtifacts/artifacts/Overrides.toml)
  • I added a link to the main README.md to point to the new artifact

@AlexisRenchon AlexisRenchon requested a review from kmdeck March 31, 2026 17:46
Comment thread era5_lake_cover_and_depth/get_era5_lake_cover_and_depth.py
@kmdeck
Copy link
Copy Markdown
Member

kmdeck commented Mar 31, 2026

looks good, but we need a link in the main README as well

@AlexisRenchon AlexisRenchon force-pushed the ar/era5_lake_cover_and_depth branch from 321fbce to deb2821 Compare April 1, 2026 15:43
@AlexisRenchon AlexisRenchon merged commit a823ce7 into main Apr 1, 2026
1 check passed
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