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/1620#issuecomment-336067289,https://api.github.com/repos/pydata/xarray/issues/1620,336067289,MDEyOklzc3VlQ29tbWVudDMzNjA2NzI4OQ==,10050469,2017-10-12T09:02:55Z,2017-10-12T09:02:55Z,MEMBER,"Closing this, as there are other more important things to deal with. the major issue here is that the important information (`bound method ImplementsArrayReduce._reduce_method..wrapped_func of`) is overwhelmed by what looks like a variable. But students should learn how to read carefully anyway ;-)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264209937 https://github.com/pydata/xarray/issues/1620#issuecomment-335895802,https://api.github.com/repos/pydata/xarray/issues/1620,335895802,MDEyOklzc3VlQ29tbWVudDMzNTg5NTgwMg==,1217238,2017-10-11T17:58:23Z,2017-10-11T17:58:23Z,MEMBER,"My guess is that NumPy's method repr is somehow due to the fact that `np.ndarray` is implemented in C. There might be some speed implication here to writing a wrapper (since it requires adding another layer of indirection) but I doubt that makes much of a difference for xarray users.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264209937 https://github.com/pydata/xarray/issues/1620#issuecomment-335707038,https://api.github.com/repos/pydata/xarray/issues/1620,335707038,MDEyOklzc3VlQ29tbWVudDMzNTcwNzAzOA==,10050469,2017-10-11T07:12:49Z,2017-10-11T07:12:49Z,MEMBER,"I guess that consistency is important too. Numpy is a bit less verbose though: ```python In [3]: a = np.linspace(1, 2) In [4]: a.min Out[4]: ``` Maybe a compromise would be more beginner friendly (e.g. `.wrapped_func of >`) but at the same time I'm not sure it's worth the trouble... ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264209937 https://github.com/pydata/xarray/issues/1620#issuecomment-335576001,https://api.github.com/repos/pydata/xarray/issues/1620,335576001,MDEyOklzc3VlQ29tbWVudDMzNTU3NjAwMQ==,1217238,2017-10-10T19:06:30Z,2017-10-11T01:43:24Z,MEMBER,"I think we could probably change this by adding a custom wrapping using descriptors, but I'm not sure that's actually a good idea. This is the standard Python repr for methods, e.g., consider: ``` >>> df = pd.DataFrame({'x': range(1000)}) >>> df.min ```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264209937 https://github.com/pydata/xarray/issues/1620#issuecomment-335461386,https://api.github.com/repos/pydata/xarray/issues/1620,335461386,MDEyOklzc3VlQ29tbWVudDMzNTQ2MTM4Ng==,10050469,2017-10-10T12:48:56Z,2017-10-10T12:48:56Z,MEMBER,Some more live news from the classroom: it is indeed very confusing for more than one student. I think the repr should be simplified.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264209937