-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Fix h5netcdf saving scalars with filters or chunks #2591
Conversation
Hello @mraspaud! Thanks for submitting the PR.
|
I'm not sure if I need to document anything as this is just a bugfix ? |
Am I right in thinking the failing travis and appveyor build aren't my doing ? |
Ok travis seems to pass, how about appveyor ? There are many time encoding errors, should it be so ? |
I believe those are unrelated to this PR. There was a series of bug fix releases of cftime following the initial 1.0.3 release (the failing build here uses 1.0.3). I think with the latest one (1.0.3.4), released a few days ago, things should be back to normal (as evidenced by the build passing in #2596). |
Ok good, I'll let this run it's course then. |
Yes, I think it would make sense to document this change under the "Bug fixes" heading. |
thanks! |
* upstream/master: Feature: N-dimensional auto_combine (pydata#2553) Support HighLevelGraphs (pydata#2603) Bump cftime version in doc environment (pydata#2604) use keep_attrs in binary operations II (pydata#2590) Temporarily mark dask-dev build as an allowed failure (pydata#2602) Fix wrong error message in interp() (pydata#2598) Add dayofyear and dayofweek accessors (pydata#2599) Fix h5netcdf saving scalars with filters or chunks (pydata#2591) Minor update to PR template (pydata#2596) Zarr consolidated (pydata#2559) fix examples (pydata#2581) Fix typo (pydata#2578) Concat docstring typo (pydata#2577) DOC: remove example using Dataset.T (pydata#2572) python setup.py test now works by default (pydata#2573) Return slices when possible from CFTimeIndex.get_loc() (pydata#2569) DOC: fix computation.rst (pydata#2567)
whats-new.rst
for all changes andapi.rst
for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)