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/1687#issuecomment-341994764,https://api.github.com/repos/pydata/xarray/issues/1687,341994764,MDEyOklzc3VlQ29tbWVudDM0MTk5NDc2NA==,10050469,2017-11-05T18:38:15Z,2017-11-05T18:38:15Z,MEMBER,"Looks good, thanks! Waiting for the tests to pass ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,271017521
https://github.com/pydata/xarray/pull/1687#issuecomment-341746579,https://api.github.com/repos/pydata/xarray/issues/1687,341746579,MDEyOklzc3VlQ29tbWVudDM0MTc0NjU3OQ==,10050469,2017-11-03T15:59:30Z,2017-11-03T15:59:30Z,MEMBER,"> @fmaussion Do you have an example of a rasterio object that would fail serialization?
No I don't, but it's more a test for the backend than for the rasterio files. See e.g. https://github.com/pydata/xarray/blob/master/xarray/backends/rasterio_.py#L154 : this line was added by @gbrener and the roundtrip tests obviously forced him to cast the bool to an int . I don't know if its good or bad ;)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,271017521
https://github.com/pydata/xarray/pull/1687#issuecomment-341740088,https://api.github.com/repos/pydata/xarray/issues/1687,341740088,MDEyOklzc3VlQ29tbWVudDM0MTc0MDA4OA==,10050469,2017-11-03T15:38:52Z,2017-11-03T15:38:52Z,MEMBER,"My reasoning (I guess) was that we shouldn't create Dataset objects which cannot be serialized to NetCDF (for whatever reason, for example because we store rasterio objects of some kind), since NetCDF is the preferred way to serialize xarray objects.
I don't mind removing these, but then the test names should be renamed too (`test_serialization_platecarree` -> `test_platecarree`)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,271017521