html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue https://github.com/pydata/xarray/pull/3527#issuecomment-554574425,https://api.github.com/repos/pydata/xarray/issues/3527,554574425,MDEyOklzc3VlQ29tbWVudDU1NDU3NDQyNQ==,14808389,2019-11-15T23:55:14Z,2019-11-15T23:55:14Z,MEMBER,"> Which behaviour can you reproduce? ```python ds = xr.Dataset( data_vars={ ""a"": ( (""x"", ""y""), [[1, 11, 26], [2, 12, 22], [3, 13, 23], [4, 16,24], [5, 15, 25]], ) }, coords={""x"": [1, 1, 1, 2, 2], ""y"": [0, 0, 1]}, ).expand_dims({""z"": [0, 1, 1, 2, 2]}) a_ds = ds.groupby(""y"").std().a a_da = ds.a.groupby(""y"").std() a_ds.identical(a_da) ``` which seems to be the same as ```python a_ds = ds.groupby(""y"").mean(""y"").a a_da = ds.a.groupby(""y"").mean(""y"") a_ds.identical(a_da) ``` My impression is that this puts the dimension grouped over to the front: - before: `ds.a.dims == (""z"", ""x"", ""y"")` - afterwards: `a_ds.dims == (""y"", ""z"", ""x"")` But I agree this discussion does not really belong here. So, thanks all!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,522523336 https://github.com/pydata/xarray/pull/3527#issuecomment-554428708,https://api.github.com/repos/pydata/xarray/issues/3527,554428708,MDEyOklzc3VlQ29tbWVudDU1NDQyODcwOA==,14808389,2019-11-15T16:24:19Z,2019-11-15T17:03:40Z,MEMBER,"I can also reproduce this with `std` Edit: I'm fairly certain this is due to what I mentioned in https://github.com/pydata/xarray/pull/3527#issuecomment-553994537","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,522523336 https://github.com/pydata/xarray/pull/3527#issuecomment-554427971,https://api.github.com/repos/pydata/xarray/issues/3527,554427971,MDEyOklzc3VlQ29tbWVudDU1NDQyNzk3MQ==,14808389,2019-11-15T16:22:41Z,2019-11-15T16:22:41Z,MEMBER,"but that's only because `mean` removes `x`, doesn't it?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,522523336 https://github.com/pydata/xarray/pull/3527#issuecomment-554404625,https://api.github.com/repos/pydata/xarray/issues/3527,554404625,MDEyOklzc3VlQ29tbWVudDU1NDQwNDYyNQ==,14808389,2019-11-15T15:29:53Z,2019-11-15T15:29:53Z,MEMBER,"well, it was surprising to me. It also might be problematic that it matters if we first pull out a `DataArray` and then compute or the other way around: ```python >>> ds = xr.Dataset( ... data_vars={ ... ""a"": ( ... (""x"", ""y""), ... [[1, 11, 26], [2, 12, 22], [3, 13, 23],^I[4, 16,^I24], [5, 15, 25]], ... ) ... }, ... coords={""x"": [1, 1, 1, 2, 2], ""y"": [0, 0, 1]}, ... ) >>> a_ds = ds.groupby(""y"").quantile(0).a >>> a_da = ds.a.groupby(""y"").quantile(0) >>> a_ds.identical(a_da) False >>> a_ds.transpose().identical(a_da) True ```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,522523336 https://github.com/pydata/xarray/pull/3527#issuecomment-554394159,https://api.github.com/repos/pydata/xarray/issues/3527,554394159,MDEyOklzc3VlQ29tbWVudDU1NDM5NDE1OQ==,14808389,2019-11-15T15:05:34Z,2019-11-15T15:05:34Z,MEMBER,"sure. But as `Dataset.quantile` and `DataArray.quantile` don't have examples either, I think it might be worth to do that in a new PR? Also, what will we do re `transpose`?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,522523336 https://github.com/pydata/xarray/pull/3527#issuecomment-553994537,https://api.github.com/repos/pydata/xarray/issues/3527,553994537,MDEyOklzc3VlQ29tbWVudDU1Mzk5NDUzNw==,14808389,2019-11-14T17:31:58Z,2019-11-14T17:58:54Z,MEMBER,"the difference is [here](https://github.com/pydata/xarray/blob/5f9ca06ef6bd4fcae3605ad8a55ad9085471bd37/xarray/core/groupby.py#L781-L783), where in `DataArrayGroupBy` the original dimension order is restored. `DatasetGroupBy` does not do that, so the test requires the `transpose`","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,522523336 https://github.com/pydata/xarray/pull/3527#issuecomment-553650563,https://api.github.com/repos/pydata/xarray/issues/3527,553650563,MDEyOklzc3VlQ29tbWVudDU1MzY1MDU2Mw==,14808389,2019-11-13T23:21:23Z,2019-11-13T23:21:23Z,MEMBER,that would open a new category. Should I use `New Features`?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,522523336 https://github.com/pydata/xarray/pull/3527#issuecomment-553647261,https://api.github.com/repos/pydata/xarray/issues/3527,553647261,MDEyOklzc3VlQ29tbWVudDU1MzY0NzI2MQ==,14808389,2019-11-13T23:10:34Z,2019-11-13T23:10:34Z,MEMBER,is `Bug Fixes` the correct section for this?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,522523336