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/7051#issuecomment-1265200159,https://api.github.com/repos/pydata/xarray/issues/7051,1265200159,IC_kwDOAMm_X85LaWwf,43316012,2022-10-03T09:51:34Z,2022-10-03T09:51:34Z,COLLABORATOR,"I think it is best to implement two functions, one for sets of dims and one for sequences of dims. This will be easier to type/read/use than trying to put both kinds into one function.
See also #7094
Maybe `parse_dims` and `parse_ordered_dims`?
","{""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1377097243
https://github.com/pydata/xarray/pull/7051#issuecomment-1255518131,https://api.github.com/repos/pydata/xarray/issues/7051,1255518131,IC_kwDOAMm_X85K1a-z,10194086,2022-09-22T20:31:13Z,2022-09-22T20:31:13Z,MEMBER,"For `xr.dot` it's also different:
```python
xr.dot(a, b, dims=None) # reduce over common dimensions
xr.dot(a, b, dims=...) # reduce over all dimensions
```
","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1377097243
https://github.com/pydata/xarray/pull/7051#issuecomment-1252964437,https://api.github.com/repos/pydata/xarray/issues/7051,1252964437,IC_kwDOAMm_X85KrrhV,5635139,2022-09-20T22:06:53Z,2022-09-20T22:06:53Z,MEMBER,"> Nothing wrong with None, it is just pythons default.
>
> The intention of `replace_none=False` was to leave None as None, which is important for some low level functions as they might be optimized (like numpy sum, which sums over all axes for None).
Not relevant to this PR but for background — it used to be that `da.groupby()` defaulted to `None`, and reduced all dimensions — now for a couple of years we require `...`.
But still `da.sum()` is equivalent to `da.sum(...)`, which is arguably a bit incongruent with `da.sum('a')` reducing _fewer_ dimensions than `da.sum(['a','b'])`. But would be quite hard to change now.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1377097243
https://github.com/pydata/xarray/pull/7051#issuecomment-1252833112,https://api.github.com/repos/pydata/xarray/issues/7051,1252833112,IC_kwDOAMm_X85KrLdY,43316012,2022-09-20T19:50:52Z,2022-09-20T19:50:52Z,COLLABORATOR,"> Yes! `...` is the better synonym — `None` is somewhat an artifact of history. So +1 for `replace_none`, which maybe we can gradually turn to `False` by default over time.
Nothing wrong with None, it is just pythons default.
The intention of `replace_none=False` was to leave None as None, which is important for some low level functions as they might be optimized (like numpy sum, which sums over all axes for None).
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1377097243
https://github.com/pydata/xarray/pull/7051#issuecomment-1252829551,https://api.github.com/repos/pydata/xarray/issues/7051,1252829551,IC_kwDOAMm_X85KrKlv,5635139,2022-09-20T19:47:20Z,2022-09-20T19:47:20Z,MEMBER,"> I always thought that these two methods are incompatible. But I guess ... Is synonym for None (= all dims) and if ... Appears in an iterable, it just means replace it with all leftover dims.
Yes! `...` is the better synonym — `None` is somewhat an artifact of history. So +1 for `replace_none`, which maybe we can gradually turn to `False` by default over time.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1377097243
https://github.com/pydata/xarray/pull/7051#issuecomment-1252586358,https://api.github.com/repos/pydata/xarray/issues/7051,1252586358,IC_kwDOAMm_X85KqPN2,43316012,2022-09-20T16:14:17Z,2022-09-20T16:14:17Z,COLLABORATOR,"> I might have thought both of them could be encapsulated in a single func... Maybe that's `infix_dims`, or fine to rename!
I always thought that these two methods are incompatible.
But I guess ... Is synonym for None (= all dims) and if ... Appears in an iterable, it just means replace it with all leftover dims.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1377097243