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
Although we have implemented transpiler logic to correctly rewrite patterns for cuDF for many cases, we still do not cover all cases, and adding \r and \n to the set of characters we use for fuzzing causes test failures.
This is quite a broad issue and will likely be split into multiple issues once more analysis has been done.
Steps/Code to reproduce bug
Add \r\n to the REGEXP_LIMITED_CHARS_COMMON constant in RegularExpressionTranspilerSuite and run the tests.
Expected behavior
Test should pass and we should fall back to CPU for anything that we cannot support correctly.
Environment details (please complete the following information)
N/A
Additional context
None
The text was updated successfully, but these errors were encountered:
Describe the bug
Although we have implemented transpiler logic to correctly rewrite patterns for cuDF for many cases, we still do not cover all cases, and adding
\r
and\n
to the set of characters we use for fuzzing causes test failures.This is quite a broad issue and will likely be split into multiple issues once more analysis has been done.
Steps/Code to reproduce bug
Add
\r\n
to theREGEXP_LIMITED_CHARS_COMMON
constant inRegularExpressionTranspilerSuite
and run the tests.Expected behavior
Test should pass and we should fall back to CPU for anything that we cannot support correctly.
Environment details (please complete the following information)
N/A
Additional context
None
The text was updated successfully, but these errors were encountered: