You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
When a numpy string is passed to parameters which take string inputs, if those parameters need to pass-through to cython layer we get a TypeError.
Expected behavior
We should not error when numpy string object is given as input. Earlier we made a generic fix to handle this incase of a np.bool_: #6426, but looks like python doesn't type-cast the np.str_ objects automatically to str objects. This is should be a trivial fix by doing str(sep) while calling the cython api, but this would have to be taken care for rest of the string parameters in other IO APIs too.
Environment overview (please complete the following information)
Environment location: [Bare-metal]
Method of cuDF install: [from source]
Environment details
Please run and paste the output of the cudf/print_env.sh script here, to gather any other relevant environment details
… io APIs (#6701)
Fixes: #6672
This PR handlesnumpy.str_ inputs given to string parameters in cuio APIs by typecasting to str before passing to cython apis.
Describe the bug
When a numpy string is passed to parameters which take string inputs, if those parameters need to pass-through to cython layer we get a
TypeError
.Steps/Code to reproduce bug
Expected behavior
We should not error when numpy string object is given as input. Earlier we made a generic fix to handle this incase of a
np.bool_
: #6426, but looks like python doesn't type-cast thenp.str_
objects automatically tostr
objects. This is should be a trivial fix by doingstr(sep)
while calling the cython api, but this would have to be taken care for rest of the string parameters in other IO APIs too.Environment overview (please complete the following information)
Environment details
Please run and paste the output of the
cudf/print_env.sh
script here, to gather any other relevant environment detailsClick here to see environment details
Additional context
Surfaced while running fuzz tests: #6001
The text was updated successfully, but these errors were encountered: