-
Notifications
You must be signed in to change notification settings - Fork 13
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
Adding units/description to netcdf files #201
Comments
Sure, I agree this would be nice, and it also should be relatively straightforward for us to implement. Initial quick take: this will require adding a 'units' attribute to the DataArray somewhere in the |
Indeed this is true; in the absence of more robust units support (e.g. with pint) there are more or less two options in this case:
|
True, but I believe that's better than no units. What about being explicit about the vertical integral operation? E.g. |
Sounds good to me! |
I started looking at this issue and was planning on adding
In this case I add the units/description to each DataArray inside of the OrderedDict? |
@micahkim23 I think we can avoid this if we add the metadata earlier in the pipeline. E.g. before this line in |
@spencerahill When I add |
@micahkim23 I think it would make the most sense if these attributes were added to only DataArrays (because the |
Currently, the descriptions/units in Var objects are not saved in the output netcdf files. It would be nice if this was the case.
The text was updated successfully, but these errors were encountered: