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 topo drag artifact #96

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

imreddyTeja
Copy link
Contributor

@imreddyTeja imreddyTeja commented Dec 23, 2024

See issue

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

@imreddyTeja imreddyTeja force-pushed the tr/add-topo-res-artifact branch from 9cc7d2a to 6baa514 Compare December 23, 2024 23:09
@imreddyTeja imreddyTeja force-pushed the tr/add-topo-res-artifact branch from 6baa514 to 59de920 Compare December 23, 2024 23:11
@imreddyTeja imreddyTeja requested a review from Sbozzolo December 23, 2024 23:11
@@ -60,6 +60,7 @@ S. Gupta et al 2022, 2024](https://github.com/CliMA/ClimaArtifacts/tree/main/soi
- [ERA5 Monthly Averages on Single Levels 1979-2024](https:////github.com/CliMA/ClimaArtifacts/tree/main/era5_monthly_averages_single_level_1979_2024)
- [Lehmann et al. 2008 Fig. 8 evaporation data](https:////github.com/CliMA/ClimaArtifacts/tree/main/lehmann2008_evaporation)
- [Forty years of ERA5 forcing data for ClimaLand](https:////github.com/CliMA/ClimaArtifacts/tree/main/forty_yrs_era5_land_forcing_data)
- [Topo Drag, created with `fregrid`](https:////github.com/CliMA/ClimaArtifacts/tree/main/topo_drag)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- [Topo Drag, created with `fregrid`](https:////github.com/CliMA/ClimaArtifacts/tree/main/topo_drag)
- [Topo Drag, created with the GDFL model](https:////github.com/CliMA/ClimaArtifacts/tree/main/topo_drag)

Comment on lines +3 to +11
This file was generated by running

```console
fregrid --standard_dimension --input_mosaic ../grid/C96_mosaic.nc --input_file topo_drag.res --interp_method conserve_order2 --remap_file ../grid/.fregrid_remap_file_288_by_180.nc --nlon 288 --nlat 180 --scalar_field t11,t21,t12,t22,hmin,hmax --output_file topo_drag.res.nc
```

The input data was obtained from the AM4 model and can be found [here](https://data1.gfdl.noaa.gov/nomads/forms/am4.0/)

The `fregrid` tool can be found [here](https://github.com/NOAA-GFDL/FRE-NCtools/tree/main)
Copy link
Member

Choose a reason for hiding this comment

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

I think this is a little misleading. Yes, fregrid is needed, but it is just a regridding tool. The important thing about this file is the input file, topo_drag.res.

@szy21 could you suggest a short description for this artifact?

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