Skip to content

Support cudf-polars str.reverse #19117

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

Merged

Conversation

brandon-b-miller
Copy link
Contributor

Closes #19025.

needs: pylibcudf tests, docs

@brandon-b-miller brandon-b-miller added feature request New feature or request non-breaking Non-breaking change labels Jun 7, 2025
@brandon-b-miller brandon-b-miller requested review from a team as code owners June 7, 2025 00:43
@github-actions github-actions bot added Python Affects Python cuDF API. CMake CMake build issue cudf-polars Issues specific to cudf-polars pylibcudf Issues specific to the pylibcudf package labels Jun 7, 2025
@GPUtester GPUtester moved this to In Progress in cuDF Python Jun 7, 2025
Copy link
Member

@mhaseeb123 mhaseeb123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving trivial cmake changes

@@ -409,6 +410,9 @@ def do_evaluate(
return Column(
plc.strings.replace.replace_multiple(column.obj, target.obj, repl.obj)
)
elif self.name is StringFunction.Name.Reverse:
(column,) = columns
return Column(plc.strings.reverse.reverse(column.obj))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just FYI for future PRs, when contructing Column could you add Column(..., dtype=self.dtype)

@brandon-b-miller
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 9c90453 into rapidsai:branch-25.08 Jun 25, 2025
90 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in cuDF Python Jun 25, 2025
@brandon-b-miller brandon-b-miller deleted the fea-cudf-polars-str-reverse branch June 25, 2025 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMake CMake build issue cudf-polars Issues specific to cudf-polars feature request New feature or request non-breaking Non-breaking change pylibcudf Issues specific to the pylibcudf package Python Affects Python cuDF API.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FEA] Support polars.Expr.str.reverse in cudf-polars
6 participants