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/5239#issuecomment-841699781,https://api.github.com/repos/pydata/xarray/issues/5239,841699781,MDEyOklzc3VlQ29tbWVudDg0MTY5OTc4MQ==,5635139,2021-05-15T17:46:01Z,2021-05-15T17:46:01Z,MEMBER,"Merging but I or @ahuang11 can do any follow-ups.
Thank you v much @ahuang11 — appreciate this was a lot of work and happy to have you contribute such a good feature.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,873519048
https://github.com/pydata/xarray/pull/5239#issuecomment-839938904,https://api.github.com/repos/pydata/xarray/issues/5239,839938904,MDEyOklzc3VlQ29tbWVudDgzOTkzODkwNA==,5635139,2021-05-12T16:54:55Z,2021-05-12T16:54:55Z,MEMBER,"@shoyer could I confirm that limiting to a single dimension satisfies this? You say ""multiple variables"" here. I mentioned a single dimension on the call but maybe wasn't clear.
> If you drop duplicates over multiple variables at once, what dimensions should the result have? In particular -- should it have the original dimensions, or should all dimensions involved be combined into one?
>
> The latter might seem a little crazy now, but would make more sense once we allow dropping over multi-dimensional variables.
>
> If we really want to avoid any possible controversy here, it might be best to stick to only supporting one variable for now, too.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,873519048
https://github.com/pydata/xarray/pull/5239#issuecomment-839925301,https://api.github.com/repos/pydata/xarray/issues/5239,839925301,MDEyOklzc3VlQ29tbWVudDgzOTkyNTMwMQ==,5635139,2021-05-12T16:33:42Z,2021-05-12T16:33:42Z,MEMBER,"@ahuang11 we discussed this on the core team call. People are excited to merge this, and appreciate you bearing with the changes.
I suggested that we narrow this even further to one dimension and merge, so we can benefit from this now and consider additions from there. Would that be OK with you?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,873519048
https://github.com/pydata/xarray/pull/5239#issuecomment-830684407,https://api.github.com/repos/pydata/xarray/issues/5239,830684407,MDEyOklzc3VlQ29tbWVudDgzMDY4NDQwNw==,5635139,2021-05-01T19:45:12Z,2021-05-01T19:45:12Z,MEMBER,"> If you drop duplicates over multiple variables at once, what dimensions should the result have? In particular -- should it have the original dimensions, or should all dimensions involved be combined into one?
>
> The latter might seem a little crazy now, but would make more sense once we allow dropping over multi-dimensional variables.
>
> If we really want to avoid any possible controversy here, it might be best to stick to only supporting one variable for now, too.
For me this case supports consistent behavior over any number of dims! Which would mean rejecting the ""stack only if you supply exactly ndims"" proposal.
Completely fine with restricting to one dim for the moment and seeing how that goes (it's always possible to pass `.drop_duplicates_coords(dim1).drop_duplicates_coords(dim2)`.
Were we going with `drop_duplicates` or `drop_duplicates_coords`? No strong view from me at all.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,873519048
https://github.com/pydata/xarray/pull/5239#issuecomment-830683784,https://api.github.com/repos/pydata/xarray/issues/5239,830683784,MDEyOklzc3VlQ29tbWVudDgzMDY4Mzc4NA==,5635139,2021-05-01T19:39:42Z,2021-05-01T19:39:42Z,MEMBER,"The docs are failing with:
```
/home/docs/checkouts/readthedocs.org/user_builds/xray/checkouts/5239/xarray/core/dataarray.py:docstring of xarray.core.dataarray.DataArray.drop_duplicates:3: WARNING: Unexpected indentation.
/home/docs/checkouts/readthedocs.org/user_builds/xray/checkouts/5239/xarray/core/dataarray.py:docstring of xarray.core.dataarray.DataArray.drop_duplicates:4: WARNING: Block quote ends without a blank line; unexpected unindent.
looking for now-outdated files... none found
```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,873519048