Message396699
Skip: If I understand right, in the patch the last two types -- float and int, will never have an effect because if float(x) and int(x) succeed, so will complex(x), and conversely, if complex(x) fails, float and int will also fail.
So the effect of the patch will be to tolerate any mix of numeric columns when the headers are textual. Which sounds fine to me, just want to confirm that sounds good to you, because the unit test in the patch is a much narrower case.
I think the test should then be something like:
a b c
1 1.1 5+0j
1.2 3+0j 4
and the code should be updated to just do `complex(x)` and remove float() and int() attempts.
Another, more strict option, - would be to special case `0` to match int, float, and complex. |
|
Date |
User |
Action |
Args |
2021-06-29 03:10:20 | andrei.avk | set | recipients:
+ andrei.avk, skip.montanaro, rhettinger, ejacq |
2021-06-29 03:10:20 | andrei.avk | set | messageid: <1624936220.12.0.516353971853.issue43625@roundup.psfhosted.org> |
2021-06-29 03:10:20 | andrei.avk | link | issue43625 messages |
2021-06-29 03:10:19 | andrei.avk | create | |
|