Skip to content

add ilamb fluxcom nee and re#158

Open
AlexisRenchon wants to merge 1 commit intomainfrom
ar/ilamb_respiration_nee
Open

add ilamb fluxcom nee and re#158
AlexisRenchon wants to merge 1 commit intomainfrom
ar/ilamb_respiration_nee

Conversation

@AlexisRenchon
Copy link
Copy Markdown
Member

@AlexisRenchon AlexisRenchon commented Apr 15, 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 April 15, 2026 16:54
]

preprocess_dict =
Dict("reco.nc" => reco_reverse_dim, "nee.nc" => nee_reverse_dim)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think it might be simpler to, instead of defining these in the other file:

reco_reverse_dim() = reverse_lat_dim("reco.nc", "reco")
nee_reverse_dim() = reverse_lat_dim("nee.nc", "nee")

just call them here

Suggested change
Dict("reco.nc" => reco_reverse_dim, "nee.nc" => nee_reverse_dim)
Dict("reco.nc" => reverse_lat_dim("reco.nc", "reco"), "nee.nc" => reverse_lat_dim("nee.nc", "nee")

[https://www.ilamb.org/ILAMB-Data/](https://www.ilamb.org/ILAMB-Data/)

### 1. `reco_FLUXCOM_reco.nc`
This `.nc` file is the FLUXCOM dataset containing observational data for ecosystem
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you note the units for both?

Copy link
Copy Markdown
Member

@kmdeck kmdeck left a comment

Choose a reason for hiding this comment

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

this looks good! do you think it makes sense to leave this as an ILAMB artifact (no specific respiration_NEE in the name) and then we can add more stuff to it if we need to? (e.g. instead of having to make a new artifact for ILAMB soil water, or ILAMB runoff)...

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