-
Notifications
You must be signed in to change notification settings - Fork 45
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
cesm_timeseries_generator.py needs to update the monthly average time coordinate for each variable #15
Labels
Comments
will need to be mindful of which variables should have this applied to them. The instantaneous variable will not need their times adjusted. So, we may need a list of variables for the tool. There may be an ncks tool that does this (confirm with Gary or Dennis) but is it efficient? Another option is to modify a python script that Kevin wrote. |
bertinia
added a commit
to bertinia/CESM_postprocessing
that referenced
this issue
Oct 26, 2017
c425638 Update __init__.py e340d4c Update README bffb962 Update setup.py REVERT: 0f5eb0b Merge pull request NCAR#34 from NCAR/devel REVERT: 34747e1 Merge pull request NCAR#29 from NCAR/devel REVERT: fa67fce Merge pull request NCAR#26 from NCAR/devel REVERT: 4ac6f44 Merge pull request NCAR#24 from NCAR/devel REVERT: 3fc272d Merge pull request NCAR#22 from NCAR/devel REVERT: 108739b Merge pull request NCAR#20 from NCAR/devel REVERT: 490ba39 Merge pull request NCAR#18 from NCAR/devel REVERT: d34c46e Merge pull request NCAR#15 from NCAR/devel REVERT: d2290b6 Merge pull request NCAR#9 from NCAR/devel REVERT: 2436413 Merge pull request NCAR#6 from NCAR-CISL-ASAP/devel REVERT: 2125a4f Merge pull request NCAR#4 from NCAR-CISL-ASAP/devel REVERT: c51daf1 Merge pull request NCAR#1 from NCAR-CISL-ASAP/devel git-subtree-dir: averager/pyAverager git-subtree-split: c425638e984b30ba743dd0dbcb30ff7de071b240
bertinia
added a commit
to bertinia/CESM_postprocessing
that referenced
this issue
Feb 15, 2018
018f31f Merge pull request NCAR#44 from NCAR/devel f28eccb Merge pull request NCAR#43 from NCAR/char_bug 6fbc661 Bump verison number 92a2b8c Merge pull request NCAR#42 from NCAR/char_bug e67cc3a Update to the previous fix: 37afc58 Merge pull request NCAR#41 from NCAR/char_bug e4b2715 Fix bug with character identification. The code was only checking for 'S1'. Compressed netcdf4 files identify characters as 'S3'. e3f121e Merge pull request NCAR#40 from NCAR/devel 28f36ec Merge pull request NCAR#38 from NCAR/devel 3563c85 Merge pull request NCAR#35 from NCAR/devel 0f5eb0b Merge pull request NCAR#34 from NCAR/devel 34747e1 Merge pull request NCAR#29 from NCAR/devel fa67fce Merge pull request NCAR#26 from NCAR/devel 4ac6f44 Merge pull request NCAR#24 from NCAR/devel 3fc272d Merge pull request NCAR#22 from NCAR/devel 108739b Merge pull request NCAR#20 from NCAR/devel 490ba39 Merge pull request NCAR#18 from NCAR/devel d34c46e Merge pull request NCAR#15 from NCAR/devel d2290b6 Merge pull request NCAR#9 from NCAR/devel 2436413 Merge pull request NCAR#6 from NCAR-CISL-ASAP/devel 2125a4f Merge pull request NCAR#4 from NCAR-CISL-ASAP/devel c51daf1 Merge pull request NCAR#1 from NCAR-CISL-ASAP/devel git-subtree-dir: averager/pyAverager git-subtree-split: 018f31ff4013fa0b240470003f27025811aeaf11
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently the history slice files and subsequent variable timeseries files reporting of the monthly
time average date is misleading to users because it writes the average value for the month at the
first time step of the subsequent month. Other models report the average value at the mid-point
of the month.
Kevin has a stand-alone serial tool that goes back and updates the files with the non-misleading time coordinate. This tool should be called from the cesm_timeseries_generator.py script as well.
The text was updated successfully, but these errors were encountered: