issue_comments

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.<locals>.wrapped_func of) is overwhelmed by what looks like a variable. But students should learn how to read carefully anyway ;-)

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.

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]: <function ndarray.min> Maybe a compromise would be more beginner friendly (e.g. <bound method ImplementsArrayReduce._reduce_method.<locals>.wrapped_func of <xarray.DataArray 'toa_sw_all_mon' (month: 12, lat: 180, lon: 360)>>) but at the same time I'm not sure it's worth the trouble...

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 <bound method DataFrame.min of x 0 0 1 1 2 2 3 3 4 4 5 5 6 6 7 7 8 8 9 9 10 10 11 11 12 12 13 13 14 14 15 15 16 16 17 17 18 18 19 19 20 20 21 21 22 22 23 23 24 24 25 25 26 26 27 27 28 28 29 29 .. ... 970 970 971 971 972 972 973 973 974 974 975 975 976 976 977 977 978 978 979 979 980 980 981 981 982 982 983 983 984 984 985 985 986 986 987 987 988 988 989 989 990 990 991 991 992 992 993 993 994 994 995 995 996 996 997 997 998 998 999 999

[1000 rows x 1 columns]> ```

Some more live news from the classroom: it is indeed very confusing for more than one student. I think the repr should be simplified.