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/818#issuecomment-220861371,https://api.github.com/repos/pydata/xarray/issues/818,220861371,MDEyOklzc3VlQ29tbWVudDIyMDg2MTM3MQ==,2443309,2016-05-22T22:47:39Z,2016-05-22T22:47:39Z,MEMBER,"@rabernat - I'm a bit late to the party here but it looks like you have gotten it straightened out. I would have suggested plotting the projected data using `Cartopy`.

@clarkfitzg - this is the exact functionality we want with 2d plot coordinates and we definitely do not want to change it.  It is a little annoying that `pcolormesh` wraps the x coordinate in the way it does but such is life.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,146182176
https://github.com/pydata/xarray/pull/818#issuecomment-220144452,https://api.github.com/repos/pydata/xarray/issues/818,220144452,MDEyOklzc3VlQ29tbWVudDIyMDE0NDQ1Mg==,2443309,2016-05-18T20:15:04Z,2016-05-18T20:15:04Z,MEMBER,"@rabernat - the `monthly-means` example was developed in an ipython notebook and then exported to `*.rst`. The [dataset](https://github.com/pydata/xarray-data/blob/master/RASM_example_data.nc) in that example doesn't have lat/lon coordinates although it should.  I'll see if I can add them this afternoon . 
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,146182176
https://github.com/pydata/xarray/pull/818#issuecomment-219875456,https://api.github.com/repos/pydata/xarray/issues/818,219875456,MDEyOklzc3VlQ29tbWVudDIxOTg3NTQ1Ng==,2443309,2016-05-17T22:38:56Z,2016-05-17T22:38:56Z,MEMBER,"@rabernat - I just had a look through the code and it looks pretty good.  I have a few broader questions though: 
1. You have a few outstanding todo items from the first comment in your PR: 
   
   > - [ ] Allow specification of which dims to stack. For example, stack in space but keep time dimension intact. (Currently it just stacks all the dimensions of the group variable.)
   > - [ ] A nice example for the docs.
   
   Where do we stand on these? You have some simple examples in the docs now but maybe you were thinking of more complete examples?
2. In https://github.com/pydata/xarray/pull/818#issuecomment-218358050, I ran into the _index is monotonic_ issue, it sounds like that was resolved.  Do we cover that case in a test?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,146182176
https://github.com/pydata/xarray/pull/818#issuecomment-218510748,https://api.github.com/repos/pydata/xarray/issues/818,218510748,MDEyOklzc3VlQ29tbWVudDIxODUxMDc0OA==,2443309,2016-05-11T16:18:05Z,2016-05-11T16:18:05Z,MEMBER,"@rabernat - See link to 2d slice with coordinates below:

[sample_for_xarray_multigroupby.nc.zip](https://github.com/pydata/xarray/files/259710/sample_for_xarray_multigroupby.nc.zip)

As for the TODO, I see now that it was there before and I agree that we should be able to side step the sorted requirement.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,146182176
https://github.com/pydata/xarray/pull/818#issuecomment-218358050,https://api.github.com/repos/pydata/xarray/issues/818,218358050,MDEyOklzc3VlQ29tbWVudDIxODM1ODA1MA==,2443309,2016-05-11T04:21:16Z,2016-05-11T04:21:16Z,MEMBER,"@rabernat - Sorry this took so long.  Comments as I play around with the new feature...
1.  I was getting some strange memory errors when trying this multidimensional groupbyon a large 4d ocean dataset (nlat: 720, nlon: 1280, time: 424, z_t: 45). my IPython Kernel just kept dying. Command was `ds.TEMP.groupby('TLONG')`. My naive guess is that this was a memory issue where a large number of bins were created - I think your first checkbox above alluded to this possibility. 
2. Operating on a 2d field (dropped `time` and `z_t` dims), I get the following error:
   
   ``` pytb
   ----> 1 da.groupby('TLAT', bins=[50, 60, 70, 80, 90])
   
   /Users/jhamman/Dropbox/src/xarray/xarray/core/common.py in groupby(self, group, squeeze, bins)
   352         if isinstance(group, basestring):
   353             group = self[group]
   --> 354         return self.groupby_cls(self, group, squeeze=squeeze, bins=bins)
   355 
   356     def rolling(self, min_periods=None, center=False, **windows):
   
   /Users/jhamman/Dropbox/src/xarray/xarray/core/groupby.py in __init__(self, obj, group, squeeze, grouper, bins)
   141             if not index.is_monotonic:
   142                 # TODO: sort instead of raising an error
   --> 143                 raise ValueError('index must be monotonic for resampling')
   144             s = pd.Series(np.arange(index.size), index)
   145             if grouper is not None:
   
   ValueError: index must be monotonic for resampling
   ```
   
   It seems this is only an issue when I specify `bins`. I see that there is a TODO statement there so maybe that will fix this.

Based on the datasets I have handy right now, I think number 2 in my list is a show show stopper so I think we want to make sure that feature makes it into this PR. 
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,146182176
https://github.com/pydata/xarray/pull/818#issuecomment-208631007,https://api.github.com/repos/pydata/xarray/issues/818,208631007,MDEyOklzc3VlQ29tbWVudDIwODYzMTAwNw==,2443309,2016-04-12T00:08:27Z,2016-04-12T00:08:27Z,MEMBER,"This looks really promising. I've gone through the code for the first time and had just a few comments.  I'll pull your branch down and give it a test drive on some real data.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,146182176
https://github.com/pydata/xarray/pull/818#issuecomment-206468443,https://api.github.com/repos/pydata/xarray/issues/818,206468443,MDEyOklzc3VlQ29tbWVudDIwNjQ2ODQ0Mw==,2443309,2016-04-06T17:09:31Z,2016-04-06T17:09:31Z,MEMBER,"@rabernat - I don't have much to add right now but I've very excited about this addition.  Once you've filled in few more of the features, ping me and I'll give it a full review and will test it out in some applications we have in house.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,146182176