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-840723034,https://api.github.com/repos/pydata/xarray/issues/5239,840723034,MDEyOklzc3VlQ29tbWVudDg0MDcyMzAzNA==,2448579,2021-05-13T17:47:47Z,2021-05-13T17:47:47Z,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. @shoyer can you kindly merge this if you're happy with it?","{""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-830508394,https://api.github.com/repos/pydata/xarray/issues/5239,830508394,MDEyOklzc3VlQ29tbWVudDgzMDUwODM5NA==,24736507,2021-05-01T03:23:37Z,2021-05-13T17:47:30Z,NONE,"Hello @ahuang11! Thanks for updating this PR. We checked the lines you've touched for [PEP 8](https://www.python.org/dev/peps/pep-0008) issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers: ##### Comment last updated at 2021-05-13 17:47:30 UTC","{""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-840264412,https://api.github.com/repos/pydata/xarray/issues/5239,840264412,MDEyOklzc3VlQ29tbWVudDg0MDI2NDQxMg==,15331990,2021-05-13T03:25:00Z,2021-05-13T03:25:00Z,CONTRIBUTOR,"Dont understand this ``` xarray/core/dataarray.py:4605: error: Argument 1 to ""isel"" of ""DataArray"" has incompatible type ""Dict[str, Any]""; expected ""Optional[Mapping[Hashable, Any]]"" [arg-type] Found 1 error in 1 file (checked 140 source files) ``` `self.isel(indexes)`","{""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-839934040,https://api.github.com/repos/pydata/xarray/issues/5239,839934040,MDEyOklzc3VlQ29tbWVudDgzOTkzNDA0MA==,15331990,2021-05-12T16:47:23Z,2021-05-12T16:47:23Z,CONTRIBUTOR,"Sure. What's the reasoning for a single dimension? On Wed, May 12, 2021, 11:34 AM Maximilian Roos ***@***.***> wrote: > @ahuang11 <https://github.com/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? > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/pydata/xarray/pull/5239#issuecomment-839925301>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/ADU7FFSS2IQ5QP7BYG4OCWLTNKUXXANCNFSM4353OVNA> > . > ","{""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-830685610,https://api.github.com/repos/pydata/xarray/issues/5239,830685610,MDEyOklzc3VlQ29tbWVudDgzMDY4NTYxMA==,14808389,2021-05-01T19:54:11Z,2021-05-01T19:54:11Z,MEMBER,"the docs build can probably be fixed by adding newlines before the section headers (""Parameters"" / ""Returns"")","{""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