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 more diurnal cycle frequencies #41

Open
forsyth2 opened this issue May 11, 2021 · 1 comment
Open

Add more diurnal cycle frequencies #41

forsyth2 opened this issue May 11, 2021 · 1 comment
Assignees
Labels
semver: new feature New feature (will increment minor version)

Comments

@forsyth2
Copy link
Collaborator

#34 adds diurnal cycle diagnostics, but only for 8xdaily (i.e., every 3 hours). We should extend zppy to handle different frequencies, such as hourly.

@forsyth2 forsyth2 added the semver: new feature New feature (will increment minor version) label May 11, 2021
@forsyth2 forsyth2 self-assigned this May 11, 2021
@golaz
Copy link
Collaborator

golaz commented May 12, 2021

Summarizing off-line discussion. Configuration file entry would look like the following for 3 hourly data:

[climo]

[[ atm_monthly_diurnal_8xdaily_180x360_aave ]]
  input_subdir = "archive/atm/hist"
  input_files = "eam.h4"
  mapping_file = /home/ac.zender/data/maps/map_ne30pg2_to_cmip6_180x360_aave.20200201.nc
  vars = "PRECT"
  frequency = "monthly_diurnal_8xdaily"

For hourly data from h5 file, change input_files and frequency:

  input_files = "eam.h5"
  frequency = "monthly_diurnal_24xdaily"

In terms of implementation:

  • Extract number of samples per day (N) from frequency string: monthly_diurnal_Nxdaily
  • Add --tbd=N to ncclimo command line arguments, for example after line
    --climatology_mode=hfc \

As @chengzhuzhang pointed out, additional changes are also needed within the e3sm_diags task. In particular, the following is not robust:

cp -s ${climoDir}/${case}.eam.h4_*_${Y1}??_${Y2}??_climo.nc .

This is kind of annoying. Since the files are in separate directories, could we just use a wildcard instead?

cp -s ${climoDir}/${case}.*_*_${Y1}??_${Y2}??_climo.nc .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: new feature New feature (will increment minor version)
Projects
None yet
Development

No branches or pull requests

2 participants