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.

Sign up for GitHub

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

Jump to bottom

Deal with CESM history file times in more generic way #22

Merged

andersy005 merged 3 commits into ncar-xdev:main from mgrover1:deal_with_time_cesm

May 27, 2021

Contributor

mgrover1 commented May 27, 2021

Helps with #21

Changes use of cf_xarray time dimension call to use T instead of time to find time variable name
Includes try-except block to grab time bound information, if it isn't there (as in CISM), feeds empty string which still allows searching for variables with time dimension, dropping time and time bound variables


          deal with times in cesm history file parser better

5d22865

mgrover1 requested a review from andersy005

May 27, 2021 15:01

mgrover1 added 2 commits

May 27, 2021 09:04


          fix whitespace issue

a4e9025


          fix trailing whitespace

8db1ddf

andersy005 approved these changes

View reviewed changes

Member

andersy005 left a comment

This looks great, @mgrover1!

andersy005 merged commit b8961bb into ncar-xdev:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet