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/1389#issuecomment-301153738,https://api.github.com/repos/pydata/xarray/issues/1389,301153738,MDEyOklzc3VlQ29tbWVudDMwMTE1MzczOA==,1217238,2017-05-12T18:36:30Z,2017-05-12T18:36:30Z,MEMBER,"> BTW, I think I messed up my co-worker's name in whats-new.rst. The spelling is right (Kyle Heuton), but the extra indentation shouldn't be there. As a result his name appears a bit out of place. Would be very much obliged if you could fix that in your next push to master. Gracias. No worries, `whats-new.rst` usually requires a fix-up before each release","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-301135310,https://api.github.com/repos/pydata/xarray/issues/1389,301135310,MDEyOklzc3VlQ29tbWVudDMwMTEzNTMxMA==,5572303,2017-05-12T17:18:49Z,2017-05-12T17:18:49Z,CONTRIBUTOR,"Thank you very much for your patience! BTW, I think I messed up my co-worker's name in `whats-new.rst`. The spelling is right (Kyle Heuton), but the extra indentation shouldn't be there. As a result his name appears a bit out of place. Would be very much obliged if you could fix that in your next push to master. Gracias.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300951935,https://api.github.com/repos/pydata/xarray/issues/1389,300951935,MDEyOklzc3VlQ29tbWVudDMwMDk1MTkzNQ==,1217238,2017-05-12T00:29:42Z,2017-05-12T00:29:42Z,MEMBER,"> api.rst should've been a checklist item :) Indeed, I will add it right now! Thanks for your contribution!!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300947995,https://api.github.com/repos/pydata/xarray/issues/1389,300947995,MDEyOklzc3VlQ29tbWVudDMwMDk0Nzk5NQ==,5572303,2017-05-11T23:59:29Z,2017-05-11T23:59:29Z,CONTRIBUTOR,`api.rst` should've been a checklist item :),"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300852820,https://api.github.com/repos/pydata/xarray/issues/1389,300852820,MDEyOklzc3VlQ29tbWVudDMwMDg1MjgyMA==,1217238,2017-05-11T17:01:30Z,2017-05-11T17:01:30Z,MEMBER,One last thing -- this needs an entry in `docs/api.rst` in order for the doc page to get built.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300850103,https://api.github.com/repos/pydata/xarray/issues/1389,300850103,MDEyOklzc3VlQ29tbWVudDMwMDg1MDEwMw==,5572303,2017-05-11T16:51:34Z,2017-05-11T16:51:34Z,CONTRIBUTOR,"If that's the case then I have nothing more to add. I've run flake8 on the individual files I touched I as well, such as `flake8 xarray/core/dataset.py`, and none of my changes are flagged.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300680230,https://api.github.com/repos/pydata/xarray/issues/1389,300680230,MDEyOklzc3VlQ29tbWVudDMwMDY4MDIzMA==,1217238,2017-05-11T04:38:19Z,2017-05-11T04:38:19Z,MEMBER,"It's very strange that fake8 complains about non-Python files. I think you can safely ignore those. On Wed, May 10, 2017 at 8:55 PM chunweiyuan wrote: > git diff upstream/master | flake8 --diff only complains about a bunch of > lines in whats-new.rst and reshaping.rst. But these complaints don't make > any sense to me. Some examples: > > doc/reshaping.rst:195:1: E112 expected an indented block > doc/reshaping.rst:203:1: E101 indentation contains mixed spaces and tabs > doc/reshaping.rst:203:1: W191 indentation contains tabs > doc/reshaping.rst:203:2: E113 unexpected indentation > > I've played around with it a bit but not seen any changes to the > complaints. What is going on? Should I even worry about the .rst files? > > — > You are receiving this because you commented. > 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}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300675689,https://api.github.com/repos/pydata/xarray/issues/1389,300675689,MDEyOklzc3VlQ29tbWVudDMwMDY3NTY4OQ==,5572303,2017-05-11T03:55:58Z,2017-05-11T03:55:58Z,CONTRIBUTOR,"` git diff upstream/master | flake8 --diff` only complains about a bunch of lines in `whats-new.rst` and `reshaping.rst`. But these complaints don't make any sense to me. Some examples: ``` doc/reshaping.rst:195:1: E112 expected an indented block doc/reshaping.rst:203:1: E101 indentation contains mixed spaces and tabs doc/reshaping.rst:203:1: W191 indentation contains tabs doc/reshaping.rst:203:2: E113 unexpected indentation ``` I've played around with it a bit but not seen any changes to the complaints. What is going on? Should I even worry about the `.rst` files?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300600778,https://api.github.com/repos/pydata/xarray/issues/1389,300600778,MDEyOklzc3VlQ29tbWVudDMwMDYwMDc3OA==,5572303,2017-05-10T20:20:12Z,2017-05-10T20:20:12Z,CONTRIBUTOR,"Haha, brilliant.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300599953,https://api.github.com/repos/pydata/xarray/issues/1389,300599953,MDEyOklzc3VlQ29tbWVudDMwMDU5OTk1Mw==,1217238,2017-05-10T20:16:49Z,2017-05-10T20:16:49Z,MEMBER,"Scroll to the top of this PR and look at your first post :) On Wed, May 10, 2017 at 11:08 AM, chunweiyuan wrote: > I just made some futile attempts to find that checklist, to no avail. Mind > helping me with a link? :) > > — > You are receiving this because you commented. > 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}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300566860,https://api.github.com/repos/pydata/xarray/issues/1389,300566860,MDEyOklzc3VlQ29tbWVudDMwMDU2Njg2MA==,5572303,2017-05-10T18:08:33Z,2017-05-10T18:08:33Z,CONTRIBUTOR,"I just made some futile attempts to find that checklist, to no avail. Mind helping me with a link? :)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300561177,https://api.github.com/repos/pydata/xarray/issues/1389,300561177,MDEyOklzc3VlQ29tbWVudDMwMDU2MTE3Nw==,1217238,2017-05-10T17:48:37Z,2017-05-10T17:48:37Z,MEMBER,"> Today is the first time I've heard of flake8. It's in the checklist for every new PR :). > Is this how you guys standardize quality checks? Yes. It would be nice to run this as a continuous integration test but we haven't set that up that. Running it once with either Python 2 or 3 is fine -- it should give the same output either way.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300555321,https://api.github.com/repos/pydata/xarray/issues/1389,300555321,MDEyOklzc3VlQ29tbWVudDMwMDU1NTMyMQ==,5572303,2017-05-10T17:30:32Z,2017-05-10T17:30:32Z,CONTRIBUTOR,"Today is the first time I've heard of flake8. Is this how you guys standardize quality checks? If so, do you run once for python 3.5 and once for 2.7?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300552112,https://api.github.com/repos/pydata/xarray/issues/1389,300552112,MDEyOklzc3VlQ29tbWVudDMwMDU1MjExMg==,1217238,2017-05-10T17:18:54Z,2017-05-10T17:18:54Z,MEMBER,@chunweiyuan have you run flake8 on this?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300534562,https://api.github.com/repos/pydata/xarray/issues/1389,300534562,MDEyOklzc3VlQ29tbWVudDMwMDUzNDU2Mg==,5572303,2017-05-10T16:16:28Z,2017-05-10T16:16:28Z,CONTRIBUTOR,Looks ready now.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300306624,https://api.github.com/repos/pydata/xarray/issues/1389,300306624,MDEyOklzc3VlQ29tbWVudDMwMDMwNjYyNA==,5572303,2017-05-09T21:27:06Z,2017-05-09T21:27:06Z,CONTRIBUTOR,Think I just copied something over from `variable.py` and changed the message a little bit...,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300292452,https://api.github.com/repos/pydata/xarray/issues/1389,300292452,MDEyOklzc3VlQ29tbWVudDMwMDI5MjQ1Mg==,1217238,2017-05-09T20:32:10Z,2017-05-09T20:32:10Z,MEMBER,"Ah, good to know. In that case, let's add a check for `dtype == object` based on `numpy.__version__` (search for `LooseVersion` for examples in the xarray codebase) and raise an informative `NotImplementedError` when necessary.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300289437,https://api.github.com/repos/pydata/xarray/issues/1389,300289437,MDEyOklzc3VlQ29tbWVudDMwMDI4OTQzNw==,5572303,2017-05-09T20:20:44Z,2017-05-09T20:20:44Z,CONTRIBUTOR," From numpy 1.11.0 release notes: ``` np.lexsort now supports arrays with object data-type The function now internally calls the generic npy_amergesort when the type does not implement a merge-sort kind of argsort method. ```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300283451,https://api.github.com/repos/pydata/xarray/issues/1389,300283451,MDEyOklzc3VlQ29tbWVudDMwMDI4MzQ1MQ==,5572303,2017-05-09T19:57:39Z,2017-05-09T19:57:39Z,CONTRIBUTOR,"1.10.4 failed with the merge sort error. 1.11.0 failed with some error that had nothing to do with merge sort (` Reason: image not found. Make sure your test modules/packages have valid Python names.`) Starting from 1.11.1 it works fine. What's the best course of action now?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300269107,https://api.github.com/repos/pydata/xarray/issues/1389,300269107,MDEyOklzc3VlQ29tbWVudDMwMDI2OTEwNw==,1217238,2017-05-09T19:02:43Z,2017-05-09T19:02:43Z,MEMBER,"> If I downgrade to numpy 1.10, I get that error. Once upgraded to 1.12, it goes away...... Does it work on numpy 1.11? We could potentially (partially) drop compatibility for older numpy releases. Or if we can identify the issue, we can raise an informative error or use a work-around.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300267772,https://api.github.com/repos/pydata/xarray/issues/1389,300267772,MDEyOklzc3VlQ29tbWVudDMwMDI2Nzc3Mg==,5572303,2017-05-09T18:57:51Z,2017-05-09T18:57:51Z,CONTRIBUTOR,"If I downgrade to numpy 1.10, I get that error. Once upgraded to 1.12, it goes away...... ![screen shot 2017-05-09 at 11 39 20 am](https://cloud.githubusercontent.com/assets/5572303/25867514/a39f64b4-34ae-11e7-992b-8e291e9849e5.png) ![screen shot 2017-05-09 at 11 39 31 am](https://cloud.githubusercontent.com/assets/5572303/25867512/a381ef56-34ae-11e7-8d5b-220161f23ddc.png) ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300030799,https://api.github.com/repos/pydata/xarray/issues/1389,300030799,MDEyOklzc3VlQ29tbWVudDMwMDAzMDc5OQ==,1217238,2017-05-09T00:52:59Z,2017-05-09T00:52:59Z,MEMBER,"Looks like one the Travis-CI tests failing on the sortby tests: https://travis-ci.org/pydata/xarray/jobs/229457729#L342 It's not immediately clear to me what's going on there, but maybe it's an issue with the old version of NumPy? The error is `TypeError: merge sort not available for item 0`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-300029141,https://api.github.com/repos/pydata/xarray/issues/1389,300029141,MDEyOklzc3VlQ29tbWVudDMwMDAyOTE0MQ==,5572303,2017-05-09T00:39:44Z,2017-05-09T00:39:44Z,CONTRIBUTOR,Perhaps it's finally ready for prime time?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-299645639,https://api.github.com/repos/pydata/xarray/issues/1389,299645639,MDEyOklzc3VlQ29tbWVudDI5OTY0NTYzOQ==,5572303,2017-05-06T15:04:31Z,2017-05-06T15:04:31Z,CONTRIBUTOR,I've added a tiny bit of extra docstring just to ease my discomfort about the NaN sort. If you feel it's redundant I can remove it.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-299230655,https://api.github.com/repos/pydata/xarray/issues/1389,299230655,MDEyOklzc3VlQ29tbWVudDI5OTIzMDY1NQ==,5572303,2017-05-04T16:01:45Z,2017-05-04T16:01:45Z,CONTRIBUTOR,Almost time to merge this baby in? :),"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-298371922,https://api.github.com/repos/pydata/xarray/issues/1389,298371922,MDEyOklzc3VlQ29tbWVudDI5ODM3MTkyMg==,1217238,2017-05-01T16:49:43Z,2017-05-01T16:49:43Z,MEMBER,"Indeed, please ignore the first CI build on Travis. For some inexplicable reason it is installing the pandas 0.20rc, which is failing due to https://github.com/pydata/xarray/issues/1386. I am slightly in favor of `sortby` but also could go either way. The PEP8 guidance on function/method naming is ""lowercase with words separated by underscores as necessary to improve readability."" I don't think an underscore is necessary for readability here.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-298365438,https://api.github.com/repos/pydata/xarray/issues/1389,298365438,MDEyOklzc3VlQ29tbWVudDI5ODM2NTQzOA==,5572303,2017-05-01T16:18:55Z,2017-05-01T16:18:55Z,CONTRIBUTOR,"Weird. Travis seems to fail in places unrelated to my changes. I'm rather agnostic to `sortby` vs. `sort_by`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227 https://github.com/pydata/xarray/pull/1389#issuecomment-298162358,https://api.github.com/repos/pydata/xarray/issues/1389,298162358,MDEyOklzc3VlQ29tbWVudDI5ODE2MjM1OA==,5635139,2017-04-29T11:05:30Z,2017-04-29T11:05:30Z,MEMBER,"We want this as `sortby`, rather than `sort_by`? Generally the latter would be more idiomatic, but if there is prior `sortby` out there, I would vote for consistency","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225222227