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/2152#issuecomment-432145295,https://api.github.com/repos/pydata/xarray/issues/2152,432145295,MDEyOklzc3VlQ29tbWVudDQzMjE0NTI5NQ==,2448579,2018-10-23T08:19:46Z,2018-10-23T08:19:46Z,MEMBER,"Failed test is cfgrib test.

Thanks @maahn ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,324105458
https://github.com/pydata/xarray/pull/2152#issuecomment-428458765,https://api.github.com/repos/pydata/xarray/issues/2152,428458765,MDEyOklzc3VlQ29tbWVudDQyODQ1ODc2NQ==,2448579,2018-10-10T06:54:07Z,2018-10-10T06:54:07Z,MEMBER,"Really sorry for the delay, @maahn. 

I've merged master, refactored out the utility functions to `utils.py`, fixed the tests locally and added a whats-new entry. I'll merge once this round of tests pass.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,324105458
https://github.com/pydata/xarray/pull/2152#issuecomment-393211218,https://api.github.com/repos/pydata/xarray/issues/2152,393211218,MDEyOklzc3VlQ29tbWVudDM5MzIxMTIxOA==,1217238,2018-05-30T15:42:39Z,2018-05-30T15:42:39Z,MEMBER,"Rather than using `interval_step_plot=True`, what about making an entirely new plot method, `plot.step()`? That would feel a little cleaner to me.

I'm not sure that a step plot would be preferred by most users for a plot over an axis on which groupby_bins was applied. Personally, I would probably prefer a line + scatter plot (e.g., `marker='s'`). Maybe @rabernat (groupby_bins author) has opinions here?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,324105458
https://github.com/pydata/xarray/pull/2152#issuecomment-391413132,https://api.github.com/repos/pydata/xarray/issues/2152,391413132,MDEyOklzc3VlQ29tbWVudDM5MTQxMzEzMg==,1217238,2018-05-23T16:31:07Z,2018-05-23T16:31:07Z,MEMBER,"A couple of other thoughts:

1. For 2D plots that already show regions (imshow/pcolormesh), I'm not sure it makes sense to update the label to include the word ""center"". This plot around shows intervals pretty clearly:
![image](https://user-images.githubusercontent.com/1217238/40438025-d3353e64-5e6b-11e8-9db8-62b9959ed4c6.png)
2. This needs tests, at least something to make sure that plotting with an Intervals on an axis does not crash in xarray/tests/test_plot.py","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,324105458
https://github.com/pydata/xarray/pull/2152#issuecomment-389939666,https://api.github.com/repos/pydata/xarray/issues/2152,389939666,MDEyOklzc3VlQ29tbWVudDM4OTkzOTY2Ng==,1197350,2018-05-17T17:07:21Z,2018-05-17T17:07:21Z,MEMBER,"This seems like a good idea. 

`groupby_bins` has a `labels` option, which can override the default labels generated by pandas. That's what is done in the (multidimensional groupby example](http://xarray.pydata.org/en/stable/examples/multidimensional-coords.html#multidimensional-groupby).

I wonder if it's worth just making this behavior the default, or generating an additional coordinate for the bin centers automatically.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,324105458