Skip to content
This repository has been archived by the owner on May 19, 2023. It is now read-only.

Fixes to work with latest cuDF #520

Merged
merged 1 commit into from
Dec 5, 2022
Merged

Fixes to work with latest cuDF #520

merged 1 commit into from
Dec 5, 2022

Conversation

efajardo-nv
Copy link
Contributor

Update CLX code after the following cuDF changes:

  • cudf.core.column.string.StringMethods.findall now returns Series instead of DataFrame
  • cudf.DataFrame.applymap was removed

@efajardo-nv efajardo-nv added bug Something isn't working non-breaking Non-breaking change labels Dec 2, 2022
@efajardo-nv efajardo-nv requested a review from a team as a code owner December 2, 2022 20:40
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@github-actions github-actions bot added the Python label Dec 2, 2022
bsuryadevara
bsuryadevara approved these changes Dec 2, 2022
View reviewed changes
Copy link
Contributor

@bsuryadevara bsuryadevara left a comment

Choose a reason for hiding this comment

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

LGTM

@efajardo-nv
Copy link
Contributor Author

@gpucibot merge

@rapids-bot rapids-bot bot merged commit ac44ee4 into rapidsai:branch-23.02 Dec 5, 2022
@efajardo-nv efajardo-nv deleted the cudf-api-updates branch December 5, 2022 18:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@BartleyR BartleyR BartleyR approved these changes

@bsuryadevara bsuryadevara bsuryadevara approved these changes

Assignees
No one assigned
Labels
bug Something isn't working non-breaking Non-breaking change Python
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@efajardo-nv @BartleyR @bsuryadevara