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/2936#issuecomment-490969141,https://api.github.com/repos/pydata/xarray/issues/2936,490969141,MDEyOklzc3VlQ29tbWVudDQ5MDk2OTE0MQ==,1217238,2019-05-09T16:13:29Z,2019-05-09T16:13:29Z,MEMBER,"Don't worry about it, this happens. If you can't figure out an immediate fix, please just mark the tests as ""xfail"" so they don't block other PRs. I'm pretty sure the issue here is in the tests, not your fix. On Thu, May 9, 2019 at 9:08 AM Martin Pletcher wrote: > Sure. I'm looking into it now. > > Sorry about that. > > — > You are receiving this because you commented. > Reply to this email directly, view it on GitHub > , or mute > the thread > > . > ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,439823329 https://github.com/pydata/xarray/pull/2936#issuecomment-490964929,https://api.github.com/repos/pydata/xarray/issues/2936,490964929,MDEyOklzc3VlQ29tbWVudDQ5MDk2NDkyOQ==,1217238,2019-05-09T16:02:07Z,2019-05-09T16:02:07Z,MEMBER,"It seems xarray (mistakenly?) relies on the immutability of index data somewhere else, and hence reuses the same pandas.Index object for data in both `data` and `coords`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,439823329 https://github.com/pydata/xarray/pull/2936#issuecomment-490963840,https://api.github.com/repos/pydata/xarray/issues/2936,490963840,MDEyOklzc3VlQ29tbWVudDQ5MDk2Mzg0MA==,1217238,2019-05-09T15:59:03Z,2019-05-09T15:59:03Z,MEMBER,"Can you take a look at the test failures on Windows? These seem to be due to your charge: ``` ================================== FAILURES =================================== ____________ TestDataArray.test_copy_coords[False-expected_orig1] _____________ self = deep = False expected_orig = array([999, 2]) Coordinates: * a (a) int32 999 2 @pytest.mark.parametrize('deep, expected_orig', [ [True, xr.DataArray(xr.IndexVariable('a', np.array([1, 2])), coords={'a': [1, 2]}, dims=['a'])], [False, xr.DataArray(xr.IndexVariable('a', np.array([999, 2])), coords={'a': [999, 2]}, dims=['a'])]]) def test_copy_coords(self, deep, expected_orig): da = xr.DataArray( np.ones([2, 2, 2]), coords={'a': [1, 2], 'b': ['x', 'y'], 'c': [0, 1]}, dims=['a', 'b', 'c']) da_cp = da.copy(deep) da_cp['a'].data[0] = 999 expected_cp = xr.DataArray( xr.IndexVariable('a', np.array([999, 2])), coords={'a': [999, 2]}, dims=['a']) > assert_identical(da_cp['a'], expected_cp) xarray\tests\test_dataarray.py:3342: _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ a = array([1, 2]) Coordinates: * a (a) int32 1 2 b = array([999, 2]) Coordinates: * a (a) int32 999 2 def assert_identical(a, b): > xarray.testing.assert_identical(a, b) E AssertionError: Left and right DataArray objects are not identical E E Differing values: E L E array([1, 2]) E R E array([999, 2]) E Differing coordinates: E L * a (a) int32 1 2 E R * a (a) int32 999 2 xarray\tests\__init__.py:195: AssertionError _____________ TestDataset.test_copy_coords[False-expected_orig1] ______________ self = deep = False expected_orig = array([999, 2]) Coordinates: * a (a) int32 999 2 @pytest.mark.parametrize('deep, expected_orig', [ [True, xr.DataArray(xr.IndexVariable('a', np.array([1, 2])), coords={'a': [1, 2]}, dims=['a'])], [False, xr.DataArray(xr.IndexVariable('a', np.array([999, 2])), coords={'a': [999, 2]}, dims=['a'])]]) def test_copy_coords(self, deep, expected_orig): ds = xr.DataArray( np.ones([2, 2, 2]), coords={'a': [1, 2], 'b': ['x', 'y'], 'c': [0, 1]}, dims=['a', 'b', 'c'], name='value').to_dataset() ds_cp = ds.copy(deep=deep) ds_cp.coords['a'].data[0] = 999 expected_cp = xr.DataArray( xr.IndexVariable('a', np.array([999, 2])), coords={'a': [999, 2]}, dims=['a']) > assert_identical(ds_cp.coords['a'], expected_cp) xarray\tests\test_dataset.py:1993: _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ a = array([1, 2]) Coordinates: * a (a) int32 1 2 b = array([999, 2]) Coordinates: * a (a) int32 999 2 def assert_identical(a, b): > xarray.testing.assert_identical(a, b) E AssertionError: Left and right DataArray objects are not identical E E Differing values: E L E array([1, 2]) E R E array([999, 2]) E Differing coordinates: E L * a (a) int32 1 2 E R * a (a) int32 999 2 xarray\tests\__init__.py:195: AssertionError ============================== warnings summary =============================== ```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,439823329