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/1796#issuecomment-356811732,https://api.github.com/repos/pydata/xarray/issues/1796,356811732,MDEyOklzc3VlQ29tbWVudDM1NjgxMTczMg==,1217238,2018-01-11T03:14:41Z,2018-01-11T03:14:41Z,MEMBER,Thanks @Zac-HD !,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-356498223,https://api.github.com/repos/pydata/xarray/issues/1796,356498223,MDEyOklzc3VlQ29tbWVudDM1NjQ5ODIyMw==,12229877,2018-01-10T04:42:44Z,2018-01-10T13:55:50Z,CONTRIBUTOR,"@shoyer makes some good points about things that should be fixed upstream in Matplotlib - namely normalization of RGB images, but I'm also going to move my color fixes (clipping instead of modulo) upstream instead. Timeline:
1. I edit this PR, it gets merged, Xarray 10.1 (ETA soonish?) can display RGB images just like Matplotlib
2. I finish my upstream PR, Matplotlib 2.2 (ETA later this month (!)) supports normalization of RGB images and fixes the modulo/clip bug
3. Once the matplotlib API is merged, I make a new PR against Xarray to support the same interface (including on older matplotlib versions)
","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-356519480,https://api.github.com/repos/pydata/xarray/issues/1796,356519480,MDEyOklzc3VlQ29tbWVudDM1NjUxOTQ4MA==,1217238,2018-01-10T07:14:39Z,2018-01-10T07:14:39Z,MEMBER,Should be fixed by https://github.com/pydata/xarray/pull/1814 which I just merged,"{""total_count"": 1, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 1, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-356519422,https://api.github.com/repos/pydata/xarray/issues/1796,356519422,MDEyOklzc3VlQ29tbWVudDM1NjUxOTQyMg==,12229877,2018-01-10T07:14:16Z,2018-01-10T07:14:16Z,CONTRIBUTOR,"Looks like `TestDatetimePlot.test_datetime_line_plot` is failing, but I'm pretty sure that's nothing to do with me (and it passed twice on Travis!).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-356498589,https://api.github.com/repos/pydata/xarray/issues/1796,356498589,MDEyOklzc3VlQ29tbWVudDM1NjQ5ODU4OQ==,1217238,2018-01-10T04:45:51Z,2018-01-10T04:45:51Z,MEMBER,"@Zac-HD Sounds good. At this point I don't think there are any blockers left for the 0.10.1 release (I wanted to get the pydap fix in, which we recently merged).","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-356272643,https://api.github.com/repos/pydata/xarray/issues/1796,356272643,MDEyOklzc3VlQ29tbWVudDM1NjI3MjY0Mw==,12229877,2018-01-09T12:38:28Z,2018-01-09T12:38:28Z,CONTRIBUTOR,"Okay... I've now seen three Travis runs. In every one there's been a pass, a fail, and three errors... but different jobs each time. At this point I'm ready to give up and wait for the Travis team to fix it 😕
The code is ready to go though! 🎉 ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-356234506,https://api.github.com/repos/pydata/xarray/issues/1796,356234506,MDEyOklzc3VlQ29tbWVudDM1NjIzNDUwNg==,10050469,2018-01-09T09:51:17Z,2018-01-09T09:51:17Z,MEMBER,"Just restarted them, let's see how it goes","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-356223193,https://api.github.com/repos/pydata/xarray/issues/1796,356223193,MDEyOklzc3VlQ29tbWVudDM1NjIyMzE5Mw==,12229877,2018-01-09T09:07:58Z,2018-01-09T09:07:58Z,CONTRIBUTOR,"More review fixes 😄
I'm pretty sure the build failure is just Travis being flaky - it passes on my machine. Could someone restart it?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-355743363,https://api.github.com/repos/pydata/xarray/issues/1796,355743363,MDEyOklzc3VlQ29tbWVudDM1NTc0MzM2Mw==,12229877,2018-01-06T12:18:56Z,2018-01-06T13:33:48Z,CONTRIBUTOR,"OK, @shoyer & @fmaussion - I think I'm done again!
The only thing I *haven't* done is change the handling of `robust=True`, as anything beyond ""scale and clip 2nd-to-98th percentiles to [0, 1]"" seems fraught with complications. If this is not what the user intended, it's at least the obvious default behaviour and should be simple to understand.
Hopefully this covers the substantial changes; but let me know if there's anything else as I'd really like this merged before the 0.10.1 release - and ideally released in time for demos at my summer school in January 😄
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-353773905,https://api.github.com/repos/pydata/xarray/issues/1796,353773905,MDEyOklzc3VlQ29tbWVudDM1Mzc3MzkwNQ==,12229877,2017-12-24T09:08:10Z,2018-01-06T12:06:44Z,CONTRIBUTOR,"Thanks for the review @shoyer! I'll do what I can in the next few days, but that might not be much at all before I get back from a no-internet camping trip around Jan 8th. Items:
- [x] add validation and tests for `rgb` argument
- [x] don't mutate the input data! (I *think* it doesn't, but obviously this needs to be checked)
- [x] simplify order heuristic, to ""use last dimension of size 3 or 4 as rgb; warn if there are multiple possible""
- [x] update rasterio example in gallery","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-355030478,https://api.github.com/repos/pydata/xarray/issues/1796,355030478,MDEyOklzc3VlQ29tbWVudDM1NTAzMDQ3OA==,10050469,2018-01-03T14:51:46Z,2018-01-03T14:51:46Z,MEMBER,"I'll add another todo (can be done in a separate PR if you don't have time ;-): modify the gallery example to plot the RGB image instead of the BW with our rasterio example [here](http://xarray.pydata.org/en/latest/auto_gallery/plot_rasterio.html#sphx-glr-auto-gallery-plot-rasterio-py).
","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-353351362,https://api.github.com/repos/pydata/xarray/issues/1796,353351362,MDEyOklzc3VlQ29tbWVudDM1MzM1MTM2Mg==,12229877,2017-12-21T13:31:28Z,2017-12-21T13:31:28Z,CONTRIBUTOR,"Done - now with optional argument and nicer commit history, Anything else?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-353260924,https://api.github.com/repos/pydata/xarray/issues/1796,353260924,MDEyOklzc3VlQ29tbWVudDM1MzI2MDkyNA==,12229877,2017-12-21T05:21:41Z,2017-12-21T05:21:41Z,CONTRIBUTOR,"Optional argument coming up then 😄
Anything else I need to do?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-353232290,https://api.github.com/repos/pydata/xarray/issues/1796,353232290,MDEyOklzc3VlQ29tbWVudDM1MzIzMjI5MA==,1217238,2017-12-21T01:32:10Z,2017-12-21T01:32:10Z,MEMBER,"I should clarify: I don't think we should *require* the explicit name for the RGB dimension, but I think we should have an argument around so that's an option if desired. This is nice because it gives better error messages when things go wrong and makes it easier to write self-documented code. This would be consistent with the rest of our visualization functions, which provide an option for being fully explicit but don't require it.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-353225923,https://api.github.com/repos/pydata/xarray/issues/1796,353225923,MDEyOklzc3VlQ29tbWVudDM1MzIyNTkyMw==,12229877,2017-12-21T00:51:35Z,2017-12-21T00:53:02Z,CONTRIBUTOR,"> I like this. My main concern is that I'd like a fully explicit way to opt into this behavior. Perhaps rgb='band' and/or rgba='band'?
I have a pretty strong preference for RGB images 'just working' - *requiring* a new argument would mean imshow has a different signature to all other plot methods without adding any information. If it's an optional argument, it barely does anything - in the rare case where the heuristic fails, you can supply `x` and `y` dims instead.
In many ways showing an RGB image is a special case, but this is the least-special-case I could make work - and it's consistent with the obvious meaning and upstream behavior of `imshow`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613
https://github.com/pydata/xarray/pull/1796#issuecomment-353093545,https://api.github.com/repos/pydata/xarray/issues/1796,353093545,MDEyOklzc3VlQ29tbWVudDM1MzA5MzU0NQ==,1217238,2017-12-20T15:27:39Z,2017-12-20T15:27:39Z,MEMBER,I like this. My main concern is that I'd like a fully explicit way to opt into this behavior. Perhaps `rgb='band'` and/or `rgba='band'`?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,283566613