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/2330#issuecomment-424499776,https://api.github.com/repos/pydata/xarray/issues/2330,424499776,MDEyOklzc3VlQ29tbWVudDQyNDQ5OTc3Ng==,5635139,2018-09-25T21:00:12Z,2018-09-25T21:00:12Z,MEMBER,"Yes, I agree - given the reluctance on the principle around ""calling a constructor on an object of the same type needs to work"" - it's right not to make this change. Thanks for holding that principle.

I'll close this; let me know if there's any change around this I make in its place.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,346294369
https://github.com/pydata/xarray/pull/2330#issuecomment-421494628,https://api.github.com/repos/pydata/xarray/issues/2330,421494628,MDEyOklzc3VlQ29tbWVudDQyMTQ5NDYyOA==,5635139,2018-09-14T21:55:33Z,2018-09-14T21:55:33Z,MEMBER,"Any votes?

I think the current version dominates the current master (i.e. it doesn't disable any useful functionality, it adds some clarity). So I would vote to merge unless anyone has any alternative suggestions (which I will gladly implement)

Thanks team","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,346294369
https://github.com/pydata/xarray/pull/2330#issuecomment-414397606,https://api.github.com/repos/pydata/xarray/issues/2330,414397606,MDEyOklzc3VlQ29tbWVudDQxNDM5NzYwNg==,5635139,2018-08-20T17:27:27Z,2018-08-20T17:27:27Z,MEMBER,"I think we should do _something_ here, rather than change behavior by coincidence.

@shoyer do you want to make the decision? 

I would vote +0.1 for the current version in the PR. V happy to be outweighed!","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,346294369
https://github.com/pydata/xarray/pull/2330#issuecomment-411461794,https://api.github.com/repos/pydata/xarray/issues/2330,411461794,MDEyOklzc3VlQ29tbWVudDQxMTQ2MTc5NA==,5635139,2018-08-08T16:08:36Z,2018-08-08T16:08:36Z,MEMBER,"@fmaussion good question

The main reason I think this is worthwhile is the change in behavior between 0.10.x and 0.11.0, when iterating over a Dataset will iterate only over its `data_vars`, and so will change `Dataset(ds)` from `ds` to `Dataset(ds.data_vars)`. 
I don't have a strong view on which option we choose, but I do think we should have a deliberate decision rather than making a breaking-change-by-coincidence.

To answer your question though, we hit this when inheriting from a Dataset: the object is initialized with a dataset, which is super-ed up to the Dataset constructor. (I realize this is narrow!). Alternatively, there is another repo that linked to the issue [here](https://github.com/jupyter-widgets/ipyleaflet/pull/178)

","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,346294369
https://github.com/pydata/xarray/pull/2330#issuecomment-411114049,https://api.github.com/repos/pydata/xarray/issues/2330,411114049,MDEyOklzc3VlQ29tbWVudDQxMTExNDA0OQ==,5635139,2018-08-07T16:14:57Z,2018-08-07T16:14:57Z,MEMBER,"> And, to confirm, you think it's OK that :

Moving ahead with this, please let me know any objections from anyone","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,346294369
https://github.com/pydata/xarray/pull/2330#issuecomment-410122566,https://api.github.com/repos/pydata/xarray/issues/2330,410122566,MDEyOklzc3VlQ29tbWVudDQxMDEyMjU2Ng==,5635139,2018-08-03T02:09:02Z,2018-08-03T02:09:02Z,MEMBER,"OK good!

And, to confirm, you think it's OK that :

`xr.Dataset(ds, coords=some_coords).coords != some_coords`","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,346294369
https://github.com/pydata/xarray/pull/2330#issuecomment-409618343,https://api.github.com/repos/pydata/xarray/issues/2330,409618343,MDEyOklzc3VlQ29tbWVudDQwOTYxODM0Mw==,5635139,2018-08-01T15:35:53Z,2018-08-01T15:35:53Z,MEMBER,"> Do we want to raise if coords / attrs are passed in addition to a Dataset? Do we want to warn for a couple versions first?

An alternative approach - more friendly but less explicit - is for any other argument (coords, attrs) supplied to overwrite the Dataset's","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,346294369