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

Add z_top to clm artifacts and rename #102

Merged
merged 1 commit into from
Feb 12, 2025
Merged

Conversation

imreddyTeja
Copy link
Contributor

@imreddyTeja imreddyTeja commented Feb 4, 2025

See accompanying ClimaLand PR

Adds Canopy height to both clm data artifacts as z_top.

The MONTHLY_MEAN_TOP is a 4D variable on Lon, lat, time, and pft.
The dominant pft is taken, and time is averaged across the months. Averaging the time
is equivalent to just selecting a time, because even though there is a time dim, there is no variance across time

In the 0.125x0.125 surface data file, the MONTHLY_MEAN_TOP is zero everywhere. Because of this,
when creating the 0.125x0.125 pft data, the script uses the closest lon and lat from MONTHLY_MEAN_TOP in the 0.9x1.25 surface data file to calculate z_top (still uses dominant PFT from 0.125x0.125 surface data file).

0.9x1.25

z_top

0.125x0.125

z_top_highres

Also rename the low res artifact from clm_data
to clm_data_0.9x1.25 and the high res artifact from
clm_data_0.125 to clm_data_0.125x0.125

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
    /groups/esm/ClimaArtifacts/artifacts/$artifact_name)
  • I added the relevant code to the Overides.toml on the Caltech Cluster
    (in /groups/esm/ClimaArtifacts/artifacts/Overrides.toml)
  • I added a link to the main README.md to point to the new artifact

Adds Canopy height to both clm data artifacts.
Also rename the low res artifact from clm_data
to clm_data_0.9x1.25 and the high res artifact from
clm_data_0.125 to clm_data_0.125x0.125
@imreddyTeja imreddyTeja force-pushed the tr/change-clm-highres-name branch from 49850e1 to 31dddcf Compare February 7, 2025 18:40
@imreddyTeja imreddyTeja changed the title Change clm_data_highres name to clm_data_0.125 Add z_top to clm artifacts and rename Feb 7, 2025
@imreddyTeja imreddyTeja marked this pull request as ready for review February 12, 2025 16:40
@imreddyTeja imreddyTeja merged commit 982e6f2 into main Feb 12, 2025
@imreddyTeja imreddyTeja deleted the tr/change-clm-highres-name branch February 12, 2025 23:58
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