-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
WIP: convert to/from cdms2 variables #236
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
Conversation
Your approach looks good to me. All people need to be able to do is move to and from cdms2 transient variables and then they'll have access to all the convenience functions defined in cdat-lite libraries like |
So I understand cdat uses masked arrays for missing values. How does it handle dates? On Mon, Sep 22, 2014 at 3:13 PM, Damien Irving [email protected]
|
I'm not sure exactly how it handles dates under the hood, however as a user before using a convenience function from the To view the time axis of a cdms2 transient variable in a convenient YYYY-MM-DD HH:MM:SS format (e.g. let's say that variable's name is Does that help? |
@DamienIrving Yes, that does help. The main thing is I'm pretty sure CDAT's climatological related functions can't handle dates in the form of a |
0cb1836
to
e919aaa
Compare
@DamienIrving I'm going to merge this now. I did add support for encoding/decoding time information, but you should probably still consider this experimental for now. Please do let me know how it works for you when you get the chance to try it out. I'll be releasing v0.3.2 with this within the next few days. |
WIP: convert to/from cdms2 variables
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Tom Nicholas <[email protected]> Co-authored-by: Anderson Banihirwe <[email protected]>
Fixes #133
@DamienIrving am I missing anything obvious here?