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

Land ERA5 forcing data artifact #5

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

kmdeck
Copy link
Member

@kmdeck kmdeck commented Apr 2, 2024

Purpose

Adds the land forcing data (1 year from ERA5 at 1 degree resolution).

To-do

update README

Content

  • create_artifacts.jl
  • output artifacts.toml information
  • File was transferred to cluster and Overrides.toml updated.

Review checklist

I have:

In the Content, I have included

  • relevant unit tests, and integration tests,
  • appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

  • I have read and checked the items on the review checklist.

@kmdeck kmdeck self-assigned this Apr 2, 2024
era5_land_forcing_data2021/Project.toml Outdated Show resolved Hide resolved
@@ -0,0 +1,2 @@
# Clima Land forcing data - ERA5 reanalysis data for 2021
more details to follow.
Copy link
Member

Choose a reason for hiding this comment

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

Can you also add a license/requested acknowledgments (if available)?

Copy link
Member Author

Choose a reason for hiding this comment

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

I will double check what I have with @braghiere

era5_land_forcing_data2021/create_artifact.jl Show resolved Hide resolved
- Leaf area index, high vegetation (lon, lat, time, "lai_hv")
- Leaf area index, low vegetation (lon, lat, time, "lai_lv")

# Attribution
Copy link
Member Author

Choose a reason for hiding this comment

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

@braghiere do you know if this is enough regarding citation of ERA5? I followed their instructions here:
https://confluence.ecmwf.int/display/CKB/Use+Case+2%3A+ERA5+hourly+data+on+single+levels+from+1940+to+present

Copy link
Member

@braghiere braghiere Apr 3, 2024

Choose a reason for hiding this comment

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

For the references of variables that looks fine. A more formal academic citation used for ERA5 is:

Hersbach, H., Bell, B., Berrisford, P., Hirahara, S., Horányi, A., Muñoz-Sabater, J., Nicolas, J., Peubey, C., Radu, R., Schepers, D., Simmons, A., Soci, C., Abdalla, S., Abellan, X., Balsamo, G., Bechtold, P., Biavati, G., Bidlot, J., Bonavita, M., … Thépaut, J.-N. (2020). The ERA5 global reanalysis. Quarterly Journal of the Royal Meteorological Society, 146(730), 1999–2049. https://doi.org/10.1002/QJ.3803

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