Skip to content

Add MAC_LWP dataset#151

Merged
ph-kev merged 1 commit intomainfrom
kp/mac-lwp
Mar 16, 2026
Merged

Add MAC_LWP dataset#151
ph-kev merged 1 commit intomainfrom
kp/mac-lwp

Conversation

@ph-kev
Copy link
Copy Markdown
Member

@ph-kev ph-kev commented Mar 11, 2026

This PR adds the Multi-Sensor Advanced Climatology of Liquid Water Path (MAC-LWP) data set. This artifact is intended to be used for calibration.

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

@ph-kev ph-kev force-pushed the kp/mac-lwp branch 2 times, most recently from 6efff75 to 77e79a4 Compare March 11, 2026 21:16
Copy link
Copy Markdown
Member

@costachris costachris left a comment

Choose a reason for hiding this comment

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

LGTM

@ph-kev ph-kev requested a review from nefrathenrici March 12, 2026 17:49
Copy link
Copy Markdown
Member

@nefrathenrici nefrathenrici left a comment

Choose a reason for hiding this comment

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

Looks good to me, I mostly caught some typos. I didn't review the testing.jl file because you told me it isn't meant to be included.

Comment thread mac_lwp/create_artifact.jl Outdated
Comment thread mac_lwp/create_artifact.jl Outdated
Comment thread mac_lwp/README.md Outdated
Comment thread mac_lwp/README.md Outdated
@ph-kev ph-kev merged commit 25a2ff9 into main Mar 16, 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.

3 participants