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/3264#issuecomment-675626899,https://api.github.com/repos/pydata/xarray/issues/3264,675626899,MDEyOklzc3VlQ29tbWVudDY3NTYyNjg5OQ==,13667821,2020-08-18T17:57:34Z,2020-08-18T17:57:34Z,CONTRIBUTOR,"@sjvrijn The new documentation looks great, with examples illustrating how it works! idxmin/idxmax is also a great addition. I think all of the concerns I raised in this issue are resolved.","{""total_count"": 1, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 1, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,485390288
https://github.com/pydata/xarray/issues/3264#issuecomment-576075198,https://api.github.com/repos/pydata/xarray/issues/3264,576075198,MDEyOklzc3VlQ29tbWVudDU3NjA3NTE5OA==,13667821,2020-01-20T01:45:59Z,2020-01-20T01:45:59Z,CONTRIBUTOR,"Actually, maybe just an if statement in ""inject_reduce_methods"" conditioned on the name of the function would work for switching to a different template. I might give this a try later.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,485390288
https://github.com/pydata/xarray/issues/3264#issuecomment-576074412,https://api.github.com/repos/pydata/xarray/issues/3264,576074412,MDEyOklzc3VlQ29tbWVudDU3NjA3NDQxMg==,13667821,2020-01-20T01:40:26Z,2020-01-20T01:41:54Z,CONTRIBUTOR,"Turns out this is more difficult than I thought... I did some digging into where these doc pages are defined (e.g. http://xarray.pydata.org/en/stable/generated/xarray.Dataset.argmin.html) and realized they are auto-generated from a template in [core/ops.py](https://github.com/pydata/xarray/blob/master/xarray/core/ops.py), which is the same for various ""reduce"" functions, most of which are more self-explanatory like max or min. Creating more useful doc page for argmin/argmax would, I believe, require writing a new template for them, which would in turn require separating argmin and argmax from ""NAN_REDUCE_METHODS"" and writing a new ""inject"" function for them. This would mess up a lot of other places that use the ""inject_reduce_methods"" function. Not sure if this is worth the effort especially since @shoyer left a todo note saying he's going to rewrite this module anyway. 

It might still be worth adding an example use case of argmax/argmin, maybe finding the dates of the highest and lowest temperatures in the [toy weather data example](http://xarray.pydata.org/en/stable/examples/weather-data.html).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,485390288