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/595#issuecomment-145161500,https://api.github.com/repos/pydata/xarray/issues/595,145161500,MDEyOklzc3VlQ29tbWVudDE0NTE2MTUwMA==,2443309,2015-10-02T21:41:09Z,2015-10-02T21:41:09Z,MEMBER,"Updated that comment. Merging now.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,109018309
https://github.com/pydata/xarray/pull/595#issuecomment-145135160,https://api.github.com/repos/pydata/xarray/issues/595,145135160,MDEyOklzc3VlQ29tbWVudDE0NTEzNTE2MA==,1217238,2015-10-02T19:37:03Z,2015-10-02T19:37:03Z,MEMBER,"Yep, works for me. Probably a good idea to add a comment to that effect to the skipped test.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,109018309
https://github.com/pydata/xarray/pull/595#issuecomment-145129683,https://api.github.com/repos/pydata/xarray/issues/595,145129683,MDEyOklzc3VlQ29tbWVudDE0NTEyOTY4Mw==,2443309,2015-10-02T19:16:01Z,2015-10-02T19:16:01Z,MEMBER,"Okay, that makes sense. Well this done. I'll merge this evening if there are no objections.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,109018309
https://github.com/pydata/xarray/pull/595#issuecomment-145118890,https://api.github.com/repos/pydata/xarray/issues/595,145118890,MDEyOklzc3VlQ29tbWVudDE0NTExODg5MA==,1217238,2015-10-02T18:39:30Z,2015-10-02T18:39:30Z,MEMBER,"@jhamman Yep, dask rewrites indexers to getitem. In the case of unsorted indexers, it currently pulls out items by block first:
https://github.com/blaze/dask/blob/de3633b2693e67f0acccebda4dfc58332aefb527/dask/array/slicing.py#L461
Wrapping all netCDF4 variables with dask.array might be a viable work around for the netCDF4-python issue, if we make dask a required dependency. We'll probably want to do that after https://github.com/blaze/dask/issues/746 is fixed anyways, though.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,109018309
https://github.com/pydata/xarray/pull/595#issuecomment-144928525,https://api.github.com/repos/pydata/xarray/issues/595,144928525,MDEyOklzc3VlQ29tbWVudDE0NDkyODUyNQ==,2443309,2015-10-02T05:34:43Z,2015-10-02T05:34:43Z,MEMBER,"@shoyer - This could use a quick review when you get a chance. For one reason or another, the `IndexError` is not raised when using `dask` so I've skipped over it.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,109018309