sha,message,author_date,committer_date,raw_author,raw_committer,repo,author,committer 851dadeb0338403e5021c3fbe80cbc9127ee672d,"No chunk warning if empty (#6402) * Test with warning when loading Zarr with empty dimension with chunks If an array has zero size (due to an empty dimension), it is saved as a single chunk regardless of Dask chunking on other dimensions (#5742). If the `chunks` parameter is provided for other dimensions when loading the Zarr file, xarray gives a warning about potentially degraded performance from splitting the single chunk. When the array has zero size, this warning seems inappropriate because: - performance degradation on an empty array should be negligible. - we don't always know if one of the dimensions is empty until loading. I would use the `chunks` parameter for dimensions that have known chunksize (to specify some multiple of that chunksize), but this only works without warning when the array is nonempty. * Don't check chunk compatibility if variable is empty/has no size * Docs describing removal of warning for `chunks` with empty array Co-authored-by: Deepak Cherian Co-authored-by: Maximilian Roos <5635139+max-sixty@users.noreply.github.com>",2022-04-09T20:27:39Z,2022-04-09T20:27:39Z,8e3ae398883b443a2f202292cd29e75d1cbcf011,cd792325681cbad9f663f2879d8b69f1edbb678f,13221727,4666753,19864447 2694046c748a51125de6d460073635f1d789958e,"Revert ""Single matplotlib import (#5794)"" (#6064) This reverts commit ea2886136dec7047186d5a73380d50130a7b5241. Co-authored-by: Illviljan <14371165+Illviljan@users.noreply.github.com>",2021-12-29T07:56:58Z,2021-12-29T07:56:58Z,8e3ae398883b443a2f202292cd29e75d1cbcf011,cd792325681cbad9f663f2879d8b69f1edbb678f,13221727,4666753,19864447 97887fd9bbfb2be58b491155c6bb08498ce294ca,"Fix saving chunked datasets with zero length dimensions (#5742) * Added test_save_emptydim for zarr backends, which fails when chunking Added test that fails when saving to zarr a dataset with a chunked array that has a dimension of length zero (Issue #5741) * Load all variables with zero entries before saving to_zarr This addresses Issue #5741 and allows `test_save_emptydim` to pass. We get around `to_zarr` not liking dask arrays with zero length dimensions by giving it numpy arrays, which works for some reason * Updated whats-new.rst with information about fix for #5741 Co-authored-by: Maximilian Roos <5635139+max-sixty@users.noreply.github.com>",2021-10-10T00:02:41Z,2021-10-10T00:02:41Z,8e3ae398883b443a2f202292cd29e75d1cbcf011,cd792325681cbad9f663f2879d8b69f1edbb678f,13221727,4666753,19864447 18e34cc8746ea2ce7be785edcaeee0bd7d4a0200,"Fix swap_dims() index names (issue #3748) (#3752) * Added test for GH3748 * Rename newly created index in swap_dims() to dim name if not multiindex Fixes GH3748 * Updated whats-new.rst with pull request information for swap_dims fix * Move tests for GH3748 into existing swap_dims tests + integrated new tests for GH3748 for DataArray into existing swap_dims tests + added similar tests for Dataset + added test for multiindex case Co-authored-by: Deepak Cherian ",2020-02-24T22:34:58Z,2020-02-24T22:34:58Z,8e3ae398883b443a2f202292cd29e75d1cbcf011,cd792325681cbad9f663f2879d8b69f1edbb678f,13221727,4666753,19864447