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/1609#issuecomment-339198772,https://api.github.com/repos/pydata/xarray/issues/1609,339198772,MDEyOklzc3VlQ29tbWVudDMzOTE5ODc3Mg==,2443309,2017-10-25T02:56:55Z,2017-10-25T02:56:55Z,MEMBER,"@shoyer - take another look. I have basically merged our two ideas and refactored the roundtrip tests. Tests are still failing but not for py2.7, on appveyor, or py3.6 locally.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,262930380 https://github.com/pydata/xarray/pull/1609#issuecomment-339005604,https://api.github.com/repos/pydata/xarray/issues/1609,339005604,MDEyOklzc3VlQ29tbWVudDMzOTAwNTYwNA==,2443309,2017-10-24T14:17:27Z,2017-10-24T14:17:27Z,MEMBER,@shoyer - ready for final review.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,262930380 https://github.com/pydata/xarray/pull/1609#issuecomment-338363428,https://api.github.com/repos/pydata/xarray/issues/1609,338363428,MDEyOklzc3VlQ29tbWVudDMzODM2MzQyOA==,2443309,2017-10-21T04:35:55Z,2017-10-21T04:35:55Z,MEMBER,Question for those who are familiar with the `scipy` backend. I have a few failing tests here on the scipy backend and I'm not really sure what's going on. It seems like it could be related to the `mmap` feature in scipy.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,262930380 https://github.com/pydata/xarray/pull/1609#issuecomment-337754946,https://api.github.com/repos/pydata/xarray/issues/1609,337754946,MDEyOklzc3VlQ29tbWVudDMzNzc1NDk0Ng==,2443309,2017-10-18T23:17:48Z,2017-10-18T23:17:48Z,MEMBER,@fmaussion - I've updated the append logic slightly. I'm wondering what you think? This version more aggressively overwrites existing variables (data_vars and coords).,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,262930380 https://github.com/pydata/xarray/pull/1609#issuecomment-335518624,https://api.github.com/repos/pydata/xarray/issues/1609,335518624,MDEyOklzc3VlQ29tbWVudDMzNTUxODYyNA==,2443309,2017-10-10T15:50:09Z,2017-10-10T15:50:09Z,MEMBER,"@fmaussion - you bring up a good point. There are two scenarios here. 1) appending to a file with existing data variables 2) appending to a file with existing coordinate variables I'm wondering if we should disallow 1, in favor of being more explicit. So: ```Python list_of_vars_to_append = ['var1', 'var2'] ds[list_of_vars_to_append].to_netcdf(filename, mode='a') # if either var1 or var2 are in filename, raise an error? ``` as apposed to the current behavior which would silently skip all vars already in `filename` and not in `list_of_vars_to_append`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,262930380