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/5984#issuecomment-979413036,https://api.github.com/repos/pydata/xarray/issues/5984,979413036,IC_kwDOAMm_X846YKgs,5635139,2021-11-25T18:24:04Z,2021-11-25T18:24:04Z,MEMBER,"> In contrast to that, I think we _can_ keep `xr.DataArray(da)` working, but _only_ if it's the only argument (i.e. I don't think we should allow `xr.DataArray(da, dims=(""x"", ""y""))` or something similar – those might still be useful, but I'd make them separate methods instead of overloading the constructor).

Perfect, very much agree, thank you for the clear description @keewis !","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1052740367
https://github.com/pydata/xarray/pull/5984#issuecomment-979262832,https://api.github.com/repos/pydata/xarray/issues/5984,979262832,IC_kwDOAMm_X846Xl1w,5635139,2021-11-25T14:31:53Z,2021-11-25T14:31:53Z,MEMBER,"> How about we raise DeprecationWarning for now asking the user to pass in `.data` instead.

Does this mean `xr.DataArray(da)` would be deprecated? That seems like a very common case. Sorry for joining late but could I ask why this is necessary, if my assumption is right? I had been thinking that is broadly equivalent to a copy.

","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1052740367