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/1869#issuecomment-364993025,https://api.github.com/repos/pydata/xarray/issues/1869,364993025,MDEyOklzc3VlQ29tbWVudDM2NDk5MzAyNQ==,6153603,2018-02-12T17:12:23Z,2018-02-12T17:12:23Z,CONTRIBUTOR,"@shoyer, I think everything is in order to apply this bug fix.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,292653302
https://github.com/pydata/xarray/pull/1869#issuecomment-362092065,https://api.github.com/repos/pydata/xarray/issues/1869,362092065,MDEyOklzc3VlQ29tbWVudDM2MjA5MjA2NQ==,6153603,2018-01-31T22:26:29Z,2018-01-31T22:26:29Z,CONTRIBUTOR,"@shoyer, do you have an opinion on whether or not to include the scipy fix or leave it out of this PR?
Also what is your opinion on the documentation? I feel nothing is needed as with this PR the code will operate as the documentation states.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,292653302
https://github.com/pydata/xarray/pull/1869#issuecomment-361766665,https://api.github.com/repos/pydata/xarray/issues/1869,361766665,MDEyOklzc3VlQ29tbWVudDM2MTc2NjY2NQ==,6153603,2018-01-30T23:11:57Z,2018-01-30T23:48:44Z,CONTRIBUTOR,"Now the tests pass (see comment beside the checkbox)
Honestly, I'm not happy with where I put the fix in order to get the tests to pass when using the scipy backend by adding an additional test, but I didn't know where else to put the test. The test to see if the variable would be checked in prepare_variables came before the encoding could be managed in encode_cf_variable in conventions.py
The alternative would be to remove the changes to scipy_.py and just have the test run only for the other engine backends and leave fixing the scipy backend for someone more familiar with it for another issue.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,292653302
https://github.com/pydata/xarray/pull/1869#issuecomment-361736359,https://api.github.com/repos/pydata/xarray/issues/1869,361736359,MDEyOklzc3VlQ29tbWVudDM2MTczNjM1OQ==,6153603,2018-01-30T21:14:52Z,2018-01-30T21:14:52Z,CONTRIBUTOR,Wait to merge... I'm still getting failed tests using the scipy backend,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,292653302
https://github.com/pydata/xarray/pull/1869#issuecomment-361734459,https://api.github.com/repos/pydata/xarray/issues/1869,361734459,MDEyOklzc3VlQ29tbWVudDM2MTczNDQ1OQ==,6153603,2018-01-30T21:07:36Z,2018-01-30T21:07:36Z,CONTRIBUTOR,"Unit tests added in 9210dfa45428842c69b8d1835013159b1230c572. The first and the third added tests use the encoding kwarg and fail without b8fd86d5614d28254406d6f9b6d4e37383efc1be applied and pass with b8fd86d5614d28254406d6f9b6d4e37383efc1be applied, so they should catch a regression.
The second added test passes both before and after b8fd86d5614d28254406d6f9b6d4e37383efc1be and was added for completeness as it pairs with the original test_explicitly_omit_fill_value, but for a coordinate.
I still think nothing needs to change in the documentation as this now follows the intent of the design.
I will leave it to you @shoyer or your delegate to address whether anything needs to be stated in the changelog / what's new.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,292653302