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/2612#issuecomment-451726877,https://api.github.com/repos/pydata/xarray/issues/2612,451726877,MDEyOklzc3VlQ29tbWVudDQ1MTcyNjg3Nw==,1217238,2019-01-06T09:13:56Z,2019-01-06T09:13:56Z,MEMBER,"Indeed, thank you @fujiisoup !","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-447651770,https://api.github.com/repos/pydata/xarray/issues/2612,447651770,MDEyOklzc3VlQ29tbWVudDQ0NzY1MTc3MA==,24736507,2018-12-16T15:28:38Z,2019-01-06T09:13:45Z,NONE,"Hello @fujiisoup! Thanks for updating the PR.
Cheers ! There are no PEP8 issues in this Pull Request. :beers:
##### Comment last updated on January 06, 2019 at 09:13 Hours UTC","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-451421221,https://api.github.com/repos/pydata/xarray/issues/2612,451421221,MDEyOklzc3VlQ29tbWVudDQ1MTQyMTIyMQ==,10050469,2019-01-04T11:36:00Z,2019-01-04T11:36:00Z,MEMBER,This is (one more time) an extremely useful addition to xarray - thanks so much @fujiisoup !,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-451350667,https://api.github.com/repos/pydata/xarray/issues/2612,451350667,MDEyOklzc3VlQ29tbWVudDQ1MTM1MDY2Nw==,1217238,2019-01-04T04:22:18Z,2019-01-04T04:22:18Z,MEMBER,I plan to merge this tomorrow unless anyone else has further suggestions.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-449220785,https://api.github.com/repos/pydata/xarray/issues/2612,449220785,MDEyOklzc3VlQ29tbWVudDQ0OTIyMDc4NQ==,2443309,2018-12-21T02:35:49Z,2018-12-21T02:35:49Z,MEMBER,"+1, on `'exact'`. I like the idea of making users be explicit about when to pad or trim edge cells.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-449214046,https://api.github.com/repos/pydata/xarray/issues/2612,449214046,MDEyOklzc3VlQ29tbWVudDQ0OTIxNDA0Ng==,1217238,2018-12-21T02:00:53Z,2018-12-21T02:00:53Z,MEMBER,"@pydata/xarray any opinions on what the default value for the `boundary` argument to `coarsen()` should be?
Personally, I like `boundary='exact'`, but I also work mostly with simulated data with dimensions setup to be exact powers of 2 :). ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-449158415,https://api.github.com/repos/pydata/xarray/issues/2612,449158415,MDEyOklzc3VlQ29tbWVudDQ0OTE1ODQxNQ==,6815844,2018-12-20T22:38:19Z,2018-12-20T22:39:20Z,MEMBER,"During writing the test codes, I realized that the default `boundary='exact'` is a little annoying.
It would be rare that the length of the data is exactly a multiple of the window size; we almost always need to type `bounary='trim'` or the other.
I personally think that `boundary='pad'` is also a good default, as it is a similar behaviour to `rolling`.
The down side of this is that it can make a uniformly spaced data an inhomogeneously spaced one.
It is something what users may not expect.
(probably `boundary='exact` is safer)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-449142367,https://api.github.com/repos/pydata/xarray/issues/2612,449142367,MDEyOklzc3VlQ29tbWVudDQ0OTE0MjM2Nw==,6815844,2018-12-20T21:33:22Z,2018-12-20T21:33:22Z,MEMBER,"Updated.
The main change is
1. API updated to
```python
dataset.coarsen(self, dim=None, boundary='exact', side='left',
coordinate_func='mean', **dim_kwargs):
```
based on comments.
2. nanmean for DateTime was implemented.
This is a kind of backward incompatible.
Previously, we raised an Error if mean is applied to a datetime array.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-448721612,https://api.github.com/repos/pydata/xarray/issues/2612,448721612,MDEyOklzc3VlQ29tbWVudDQ0ODcyMTYxMg==,1217238,2018-12-19T19:48:02Z,2018-12-19T19:48:02Z,MEMBER,"I do think it would be nice for `resample()` to work with numbers and along multiple dimensions, but that's definitely a bigger project.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-448719698,https://api.github.com/repos/pydata/xarray/issues/2612,448719698,MDEyOklzc3VlQ29tbWVudDQ0ODcxOTY5OA==,5635139,2018-12-19T19:41:53Z,2018-12-19T19:41:53Z,MEMBER,"Perfect, thanks for pointing that out (I've generally been a bit out of the loop recently...)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-448718248,https://api.github.com/repos/pydata/xarray/issues/2612,448718248,MDEyOklzc3VlQ29tbWVudDQ0ODcxODI0OA==,1217238,2018-12-19T19:37:09Z,2018-12-19T19:37:09Z,MEMBER,"@max-sixty we discussed this a little bit: https://github.com/pydata/xarray/issues/2525#issuecomment-434477550
The main difference is that resample is coordinate based, whereas this is integer position based which makes the implementation considerably simpler.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-448717771,https://api.github.com/repos/pydata/xarray/issues/2612,448717771,MDEyOklzc3VlQ29tbWVudDQ0ODcxNzc3MQ==,5635139,2018-12-19T19:35:29Z,2018-12-19T19:35:29Z,MEMBER,"> I think it's multi-dimensional resampling / rolling. resample & rolling operate one dimension at a time.
Thanks. If this is multi-dimensional resampling / rolling, could we implement this functionality within those methods, enabling multiple dimensions?
Potentially the implementations are different enough that the arguments don't have enough overlap?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-447924905,https://api.github.com/repos/pydata/xarray/issues/2612,447924905,MDEyOklzc3VlQ29tbWVudDQ0NzkyNDkwNQ==,2448579,2018-12-17T17:17:18Z,2018-12-17T17:17:18Z,MEMBER,@max-sixty I think it's multi-dimensional resampling / rolling. resample & rolling operate one dimension at a time.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-447912054,https://api.github.com/repos/pydata/xarray/issues/2612,447912054,MDEyOklzc3VlQ29tbWVudDQ0NzkxMjA1NA==,5635139,2018-12-17T16:41:58Z,2018-12-17T16:41:58Z,MEMBER,Forgive me if this has an obvious answer: to what extent is this downsampling? Could this be done with `resample`?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-447736226,https://api.github.com/repos/pydata/xarray/issues/2612,447736226,MDEyOklzc3VlQ29tbWVudDQ0NzczNjIyNg==,6815844,2018-12-17T06:27:35Z,2018-12-17T06:27:35Z,MEMBER,"@dcherian
Thanks.
> (da.time - da.time[0]).mean() + da.time[0]
I would add mean for datetime-arrays.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755
https://github.com/pydata/xarray/pull/2612#issuecomment-447664380,https://api.github.com/repos/pydata/xarray/issues/2612,447664380,MDEyOklzc3VlQ29tbWVudDQ0NzY2NDM4MA==,2448579,2018-12-16T18:18:55Z,2018-12-16T18:18:55Z,MEMBER,"`(da.time - da.time[0]).mean() + da.time[0]`
Or we could default to `min()` and add an `loffset` kwarg like `resample` (https://pandas.pydata.org/pandas-docs/stable/generated/pandas.DataFrame.resample.html)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,391477755