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/issues/4727#issuecomment-751843789,https://api.github.com/repos/pydata/xarray/issues/4727,751843789,MDEyOklzc3VlQ29tbWVudDc1MTg0Mzc4OQ==,5635139,2020-12-28T19:50:17Z,2020-12-28T19:50:17Z,MEMBER,"I agree with @toddrjen that the words of `assert_identical` strongly suggest that the dtypes would be checked. I would suggest we either: - Fail on different dtypes in 0.17 - If there are lots of breaks / downstream libraries that may break: add a `FutureWarning` on different dtypes and change in the future Happy to help on this!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,773750763 https://github.com/pydata/xarray/issues/4727#issuecomment-750926206,https://api.github.com/repos/pydata/xarray/issues/4727,750926206,MDEyOklzc3VlQ29tbWVudDc1MDkyNjIwNg==,10194086,2020-12-24T16:51:42Z,2020-12-24T19:28:23Z,MEMBER,"There seems to be a subtle difference between comparing `Dataset` and `DataArray`. `DataArray` compares `coords`: https://github.com/pydata/xarray/blob/03d8d56c9b6d090f0de2475202368b08435eaeb5/xarray/core/dataarray.py#L2792 while `Dataset` compares `_variables`: https://github.com/pydata/xarray/blob/03d8d56c9b6d090f0de2475202368b08435eaeb5/xarray/core/dataset.py#L1457-L1459 Thus once we compare `xarray.core.variable.IndexVariable` and once we compare `xarray.core.dataarray.DataArray` and the comparison takes different code paths... ```python import xarray as xr air = xr.tutorial.open_dataset(""air_temperature"") type(air._variables[""lat""]) # xarray.core.variable.IndexVariable type(air.air.coords[""lat""]) # xarray.core.dataarray.DataArray ``` --- edit: typo ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,773750763 https://github.com/pydata/xarray/issues/4727#issuecomment-750386723,https://api.github.com/repos/pydata/xarray/issues/4727,750386723,MDEyOklzc3VlQ29tbWVudDc1MDM4NjcyMw==,14808389,2020-12-23T16:56:11Z,2020-12-23T17:04:12Z,MEMBER,this has come up in https://github.com/pydata/xarray/pull/3706#issuecomment-583259053 before.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,773750763 https://github.com/pydata/xarray/issues/4727#issuecomment-750354606,https://api.github.com/repos/pydata/xarray/issues/4727,750354606,MDEyOklzc3VlQ29tbWVudDc1MDM1NDYwNg==,2448579,2020-12-23T15:40:07Z,2020-12-23T15:40:07Z,MEMBER,"+1 on adding `check_dtype`. I think it should be default `False` for `assert_equal`. It could be default `True` for `assert_identical` but this would be very backward incompatible.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,773750763