-
Notifications
You must be signed in to change notification settings - Fork 89
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
Missing netcdf4 library #435
Comments
Hi @rogersamso netcdf4 library is in the optional dependencies list. I would keep it there so as not to increment the dependencies as most people may not use features that need it (serialize_external_objects and dataset output). I will, in any case, include in the documentation that this function requires netcdf4 library installation. |
maybe a try except around the ds.to_netcdf would also be helpful, as the error message that it sends is not clear at all. |
Yes, that will be nice. Could you take care of it, please? |
I added info about the requirement in the installation.rst and functions docstring in the PR that will be merged tomorrow. Please let me know if you want to include the error message. |
I think it would be useful, but I don't have the time. So do what you think is best. |
The netcdf4 library is required to serialize external objects to netcdf files with the
serialize_external_objects
method from theMacro
class.The text was updated successfully, but these errors were encountered: