Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Numba 0.57 will break row UDFs #13002

Closed
gmarkall opened this issue Mar 23, 2023 · 1 comment
Closed

[BUG] Numba 0.57 will break row UDFs #13002

gmarkall opened this issue Mar 23, 2023 · 1 comment
Labels
0 - Blocked Cannot progress due to external reasons bug Something isn't working Python Affects Python cuDF API.

Comments

@gmarkall
Copy link
Contributor

Describe the bug

Identified by @brandon-b-miller:

numba/numba#8120 added a bitwidth attribute to the Record type, which is computed by converting the fields to their NumPy dtype and getting the item size. However, cuDF uses Record instances that have non-NumPy members - I believe we generate these types here: https://github.com/rapidsai/cudf/blob/branch-23.04/python/cudf/cudf/core/udf/row_function.py#L30

The consequence is that Numba raises when attempting to convert these members to NumPy dtypes.

Steps/Code to reproduce bug

Running the test_string_udf_strip test should reproduce the issue when Numba built from its main branch is in the environment should cause this: i.e. pytest test_udf_masked_ops.py -k test_string_udf_strip

Expected behavior

Successful execution of the test.

Environment overview (please complete the following information)

Any environment in which Numba built from current main is present (or the 0.57 RC / release when it's out).

Environment details

Not supplied.

Additional context

None.

@gmarkall gmarkall added bug Something isn't working Needs Triage Need team to review and classify labels Mar 23, 2023
@gmarkall gmarkall mentioned this issue Mar 23, 2023
3 tasks
@GregoryKimball GregoryKimball added Python Affects Python cuDF API. 0 - Blocked Cannot progress due to external reasons and removed Needs Triage Need team to review and classify labels Jun 7, 2023
@vyasr
Copy link
Contributor

vyasr commented May 15, 2024

Resolved in #13271

@vyasr vyasr closed this as completed May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - Blocked Cannot progress due to external reasons bug Something isn't working Python Affects Python cuDF API.
Projects
None yet
Development

No branches or pull requests

3 participants