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/1453#issuecomment-318717311,https://api.github.com/repos/pydata/xarray/issues/1453,318717311,MDEyOklzc3VlQ29tbWVudDMxODcxNzMxMQ==,1217238,2017-07-28T17:39:11Z,2017-07-28T17:39:11Z,MEMBER,Thanks @deeplycloudy !,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235761029
https://github.com/pydata/xarray/pull/1453#issuecomment-316176226,https://api.github.com/repos/pydata/xarray/issues/1453,316176226,MDEyOklzc3VlQ29tbWVudDMxNjE3NjIyNg==,1217238,2017-07-18T19:44:48Z,2017-07-18T19:44:48Z,MEMBER,"Sounds good, I'll take another look shortly.

On Tue, Jul 18, 2017 at 10:46 AM, Joe Hamman <notifications@github.com>
wrote:

> *@jhamman* approved this pull request.
>
> I think we're good to go here. @shoyer <https://github.com/shoyer> - I'll
> let you have the final review/merge.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/pydata/xarray/pull/1453#pullrequestreview-50689714>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABKS1nsL9Rg-WevNBgY8i30AFt5SJGuBks5sPO-FgaJpZM4N5YIf>
> .
>
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235761029
https://github.com/pydata/xarray/pull/1453#issuecomment-315620355,https://api.github.com/repos/pydata/xarray/issues/1453,315620355,MDEyOklzc3VlQ29tbWVudDMxNTYyMDM1NQ==,1325771,2017-07-16T16:29:10Z,2017-07-16T16:29:10Z,CONTRIBUTOR,"Tests now pass after I realized I wasn't converting the _FillValue to unsigned. 

I also turned off PyNIO's internal support for masking, in keeping with the philosophy that `xarray` should only use the backends to retrieve the bytes as represented on disk.

Note that some of the CI builds are skipping most of their tests (e.g, py=3.4; you can tell by the run time). This is a problem in other PRs as well.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235761029
https://github.com/pydata/xarray/pull/1453#issuecomment-309767695,https://api.github.com/repos/pydata/xarray/issues/1453,309767695,MDEyOklzc3VlQ29tbWVudDMwOTc2NzY5NQ==,1325771,2017-06-20T14:11:36Z,2017-06-20T14:11:36Z,CONTRIBUTOR,"The CI fail is for 2.7/cdat/pynio in a couple of my new tests. In one, the fill value is not being applied, while in the other the unsigned conversion isn't happening. Are there any known differences in that cdat/pynio stack that would cause these to fail while others pass?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235761029
https://github.com/pydata/xarray/pull/1453#issuecomment-309623307,https://api.github.com/repos/pydata/xarray/issues/1453,309623307,MDEyOklzc3VlQ29tbWVudDMwOTYyMzMwNw==,1325771,2017-06-20T02:03:12Z,2017-06-20T02:03:12Z,CONTRIBUTOR,I've created a new `UnsignedIntTypeArray` and have separated the logic from `mask_and_scale`. Lint has been removed and docs updated.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235761029
https://github.com/pydata/xarray/pull/1453#issuecomment-308598903,https://api.github.com/repos/pydata/xarray/issues/1453,308598903,MDEyOklzc3VlQ29tbWVudDMwODU5ODkwMw==,1217238,2017-06-15T00:59:25Z,2017-06-15T00:59:25Z,MEMBER,"Instead of putting this alongside the `mask_and_scale` logic, can you make a separate class to do the dtype fixing in `decode_cf_variable`? Take a look at `BoolTypeArray` for an example.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235761029
https://github.com/pydata/xarray/pull/1453#issuecomment-308326393,https://api.github.com/repos/pydata/xarray/issues/1453,308326393,MDEyOklzc3VlQ29tbWVudDMwODMyNjM5Mw==,1325771,2017-06-14T05:49:52Z,2017-06-14T05:49:52Z,CONTRIBUTOR,"In addition to the included (basic) test I've also tested this with the real-world data that motivated the PR and #1444. While it's a working draft, I'd welcome comments on the basic approach and appropriateness of the test coverage.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235761029