FIX: let pandas IndexInt64 work for boxplot#13398
Conversation
jklymak
force-pushed
the
fix-boxplot-pandas
branch
from
1e7caa0 to
28bc931
Compare
|
I think this fix is "OK" as a stopgap, but I think what really should happen is |
| positions = list(range(1, N + 1)) | ||
| elif len(positions) != N: | ||
| raise ValueError(datashape_message.format("positions")) | ||
| positions = np.array(positions) |
There was a problem hiding this comment.
This is a good fix for the case described in the ticket, but I think the code will continue raising obscure error when positions are not numbers. Can you add a check after this line about the dtype of the array being a number? Maybe something like:
if not isinstance(positions.dtype, numbers.Number):
raise TypeError("positions should be a iterable of dtype number, but found dtype %s" % blah blah blah.)When we end up supporting something else than numbers, this can be updated.
There was a problem hiding this comment.
I couldn't quite get that check to work; I had to check the first element. But if there is a better way, let me know.
jklymak
force-pushed
the
fix-boxplot-pandas
branch
from
ef7afc4 to
728106a
Compare
|
Thanks! |
4 participants
PR Summary
Closes #10022 by casting the list of positions to an array as suggested by @tacaswell
PR Checklist