Skip to content

Restore column type metadata with dropna to fix factorize API - #13980

Merged
rapids-bot[bot] merged 1 commit into
NVIDIA:branch-23.10from
galipremsagar:13979
Aug 28, 2023
Merged

Restore column type metadata with dropna to fix factorize API#13980
rapids-bot[bot] merged 1 commit into
NVIDIA:branch-23.10from
galipremsagar:13979

Conversation

@galipremsagar

Copy link
Copy Markdown
Contributor

Description

closes #13979

This PR restores column type metadata for dropna call, absense of this restoration was causing an issue with the CategoricalColumn.dropna that was necessary for factorize API.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@galipremsagar galipremsagar added bug Something isn't working 3 - Ready for Review Ready for review by team Python Affects Python cuDF API. 4 - Needs cuDF (Python) Reviewer non-breaking Non-breaking change labels Aug 28, 2023
@galipremsagar galipremsagar self-assigned this Aug 28, 2023
@galipremsagar
galipremsagar requested a review from a team as a code owner August 28, 2023 05:58
@galipremsagar
galipremsagar requested review from bdice and wence- August 28, 2023 05:58
@galipremsagar galipremsagar added 5 - Ready to Merge Testing and reviews complete, ready to merge and removed 3 - Ready for Review Ready for review by team 4 - Needs cuDF (Python) Reviewer labels Aug 28, 2023
@galipremsagar

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit d138dd0 into NVIDIA:branch-23.10 Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to Merge Testing and reviews complete, ready to merge bug Something isn't working non-breaking Non-breaking change Python Affects Python cuDF API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] factorize returning incorrect results for CategoricalDtype

2 participants