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/1802#issuecomment-354469859,https://api.github.com/repos/pydata/xarray/issues/1802,354469859,MDEyOklzc3VlQ29tbWVudDM1NDQ2OTg1OQ==,3698640,2017-12-29T16:45:37Z,2017-12-29T16:45:37Z,CONTRIBUTOR,Ok this is good to go if you all do want to enable _FillValue for variable-length unicode strings with a netCDF4 backend. Seems like there's a lot of prior work/thinking in this space though so no worries if you want to wait.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,285006452
https://github.com/pydata/xarray/pull/1802#issuecomment-354379246,https://api.github.com/repos/pydata/xarray/issues/1802,354379246,MDEyOklzc3VlQ29tbWVudDM1NDM3OTI0Ng==,3698640,2017-12-29T00:22:57Z,2017-12-29T00:22:57Z,CONTRIBUTOR,lol. no I'm just walking around in your footsteps @shoyer. I've just enabled the tests you presumably wrote for #1647 & #1648. Curious why variable-length unicode strings with _FillHoles using netCDF4 doesn't currently work in master?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,285006452
https://github.com/pydata/xarray/pull/1802#issuecomment-354378158,https://api.github.com/repos/pydata/xarray/issues/1802,354378158,MDEyOklzc3VlQ29tbWVudDM1NDM3ODE1OA==,3698640,2017-12-29T00:11:53Z,2017-12-29T00:11:53Z,CONTRIBUTOR,"hmm. Seems I'm touching on a much larger issue here: Unidata/netcdf4-python#730

The round-trip works for me using a netcdf4 engine once this fix is implemented in conventions.py. There are tests that are ready to demonstrate this in test_backends.py:836-843, but running these tests (by removing the `pytest.raises` lines) applies to both netCDF4 and h5netcdf backends.

Should these use cases be split up?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,285006452