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/3336#issuecomment-566861922,https://api.github.com/repos/pydata/xarray/issues/3336,566861922,MDEyOklzc3VlQ29tbWVudDU2Njg2MTkyMg==,5635139,2019-12-18T04:27:11Z,2019-12-18T04:27:11Z,MEMBER,"@gwgundersen I'm really sorry to have kept you waiting, especially after this PR and the others strong ones you've recently done. I temporarily paused my xarray oversight while I came up to speed with a new job and let this drop.

@dcherian any chance you could take a glance? Otherwise I'll look properly this weekend. Thanks.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,497416198
https://github.com/pydata/xarray/pull/3336#issuecomment-536231394,https://api.github.com/repos/pydata/xarray/issues/3336,536231394,MDEyOklzc3VlQ29tbWVudDUzNjIzMTM5NA==,5635139,2019-09-28T22:55:00Z,2019-09-28T22:55:00Z,MEMBER,"Another alternative is a decorator that can encapsulate some of this meta-programming (could also do the work of `either_dict_or_kwargs`)

But if we're only doing this for a couple of methods, I would advocate we just write out the keywords. I know it's a bit of a inelegant defeat, but it's pythonic. 

`sel` & `isel` are probably more important than the others

@shoyer do you have thoughts re encapsulation vs magic?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,497416198
https://github.com/pydata/xarray/pull/3336#issuecomment-536231119,https://api.github.com/repos/pydata/xarray/issues/3336,536231119,MDEyOklzc3VlQ29tbWVudDUzNjIzMTExOQ==,5635139,2019-09-28T22:48:35Z,2019-09-28T22:48:35Z,MEMBER,">  I did a basic performance test, and locals() is 1/100th of a second faster on average.

Do you know what the absolute numbers are? i.e. how much performance penalty does the faster routine introduce?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,497416198
https://github.com/pydata/xarray/pull/3336#issuecomment-534730840,https://api.github.com/repos/pydata/xarray/issues/3336,534730840,MDEyOklzc3VlQ29tbWVudDUzNDczMDg0MA==,5635139,2019-09-24T20:20:08Z,2019-09-24T20:20:08Z,MEMBER,"That's a clever approach. It's a bit magic, but maybe the right tradeoff. Are there any performance implications?

I agree with @gwgundersen re writing each item out being impractical, unless we're only doing for `sel` & `isel`. 

Any broader thoughts re the need to apply to all methods, vs this sort of magic?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,497416198