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/issues/3280#issuecomment-532263016,https://api.github.com/repos/pydata/xarray/issues/3280,532263016,MDEyOklzc3VlQ29tbWVudDUzMjI2MzAxNg==,2448579,2019-09-17T15:05:23Z,2019-09-17T15:05:23Z,MEMBER,#3313 and #3314 should be looked over by someone who knows the `auto_combine` and `groupby` code.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,489270698 https://github.com/pydata/xarray/issues/3280#issuecomment-531361035,https://api.github.com/repos/pydata/xarray/issues/3280,531361035,MDEyOklzc3VlQ29tbWVudDUzMTM2MTAzNQ==,1217238,2019-09-13T19:23:00Z,2019-09-13T19:23:00Z,MEMBER,"Yeah, there's also some question about if combine_nested() and combine_by_coords() really solve all the use-cases of auto_combine(). For example, a bunch of files that don't contain the concatenated dimension and have different variables can be combined automatically by auto_combine() but not combine_by_coord(). On Fri, Sep 13, 2019 at 9:03 AM Deepak Cherian wrote: > We were also planing to deprecate auto_combine for 0.13 but we've been > warning about that only since 29 June, 2019 so that should be pushed to > 0.14. I've added this to the list. > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > , > or mute the thread > > . > ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,489270698 https://github.com/pydata/xarray/issues/3280#issuecomment-531295841,https://api.github.com/repos/pydata/xarray/issues/3280,531295841,MDEyOklzc3VlQ29tbWVudDUzMTI5NTg0MQ==,2448579,2019-09-13T16:03:17Z,2019-09-13T16:03:17Z,MEMBER,"We were also planing to deprecate `auto_combine` for 0.13 but we've been warning about that only since 29 June, 2019 so that should be pushed to 0.14. I've added this to the list.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,489270698 https://github.com/pydata/xarray/issues/3280#issuecomment-529523741,https://api.github.com/repos/pydata/xarray/issues/3280,529523741,MDEyOklzc3VlQ29tbWVudDUyOTUyMzc0MQ==,10194086,2019-09-09T15:10:55Z,2019-09-09T15:10:55Z,MEMBER,"Yes, fine for me. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,489270698 https://github.com/pydata/xarray/issues/3280#issuecomment-529520452,https://api.github.com/repos/pydata/xarray/issues/3280,529520452,MDEyOklzc3VlQ29tbWVudDUyOTUyMDQ1Mg==,2448579,2019-09-09T15:04:26Z,2019-09-09T15:04:35Z,MEMBER,"> > stop supporting a list of colors in the `cmap` argument > > I am a bit worried about the 3rd last item on your list. I currently still rely on this behavior because the alternative does not work for me, see #3284. @mathause Does your PR mean that we can go forward with the deprecation then?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,489270698 https://github.com/pydata/xarray/issues/3280#issuecomment-529366898,https://api.github.com/repos/pydata/xarray/issues/3280,529366898,MDEyOklzc3VlQ29tbWVudDUyOTM2Njg5OA==,5635139,2019-09-09T08:54:23Z,2019-09-09T08:54:23Z,MEMBER,I think all the 'must-dos' are done,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,489270698 https://github.com/pydata/xarray/issues/3280#issuecomment-529197236,https://api.github.com/repos/pydata/xarray/issues/3280,529197236,MDEyOklzc3VlQ29tbWVudDUyOTE5NzIzNg==,5635139,2019-09-08T12:17:19Z,2019-09-08T12:17:19Z,MEMBER,"@shoyer I edited your comment to check-off the ones in #3292, hope that's OK. (would be good to have a system from GH that didn't involve free editing of others' comments!)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,489270698 https://github.com/pydata/xarray/issues/3280#issuecomment-528950471,https://api.github.com/repos/pydata/xarray/issues/3280,528950471,MDEyOklzc3VlQ29tbWVudDUyODk1MDQ3MQ==,1217238,2019-09-06T17:49:28Z,2019-09-06T17:49:28Z,MEMBER,"> > stop supporting a list of colors in the `cmap` argument > > I am a bit worried about the 3rd last item on your list. I currently still rely on this behavior because the alternative does not work for me, see #3284. OK, we should put this off until the alternative works! But I do think we want to stop overloading `cmap` in an incompatible way.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,489270698 https://github.com/pydata/xarray/issues/3280#issuecomment-528787208,https://api.github.com/repos/pydata/xarray/issues/3280,528787208,MDEyOklzc3VlQ29tbWVudDUyODc4NzIwOA==,10194086,2019-09-06T09:39:51Z,2019-09-06T09:39:51Z,MEMBER,"> stop supporting a list of colors in the `cmap` argument I am a bit worried about the 3rd last item on your list. I currently still rely on this behavior because the alternative does not work for me, see #3284. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,489270698