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

Raise error in reindex when index is not unique #14400

Merged
merged 8 commits into from
Nov 15, 2023

Conversation

galipremsagar
Copy link
Contributor

@galipremsagar galipremsagar commented Nov 14, 2023

Description

Fixes: #14398
This PR raises an error in reindex API when reindexing is performed on a non-unique index column.

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 breaking Breaking change labels Nov 14, 2023
@galipremsagar galipremsagar self-assigned this Nov 14, 2023
@galipremsagar galipremsagar requested review from a team as code owners November 14, 2023 00:10
Copy link
Contributor

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

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

cudf changes look good

@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 Dask Reviewer labels Nov 15, 2023
@galipremsagar
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 8deb3dd into rapidsai:branch-23.12 Nov 15, 2023
65 checks passed
galipremsagar added a commit to galipremsagar/cudf that referenced this pull request Nov 16, 2023
Fixes: rapidsai#14398 
This PR raises an error in `reindex` API when reindexing is performed on a non-unique index column.

Authors:
  - GALI PREM SAGAR (https://github.com/galipremsagar)

Approvers:
  - Matthew Roeschke (https://github.com/mroeschke)
  - Lawrence Mitchell (https://github.com/wence-)

URL: rapidsai#14400
raydouglass pushed a commit that referenced this pull request Nov 16, 2023
Bacport of #14400 
Fixes: #14398 
This PR raises an error in `reindex` API when reindexing is performed on a non-unique index column.

Authors:
  - GALI PREM SAGAR (https://github.com/galipremsagar)

Approvers:
  - Matthew Roeschke (https://github.com/mroeschke)
  - Lawrence Mitchell (https://github.com/wence-)

URL: #14400

Authors:
   - GALI PREM SAGAR (https://github.com/galipremsagar)

Approvers:
   - Richard (Rick) Zamora (https://github.com/rjzamora)
   - Ashwin Srinath (https://github.com/shwina)
   - Ray Douglass (https://github.com/raydouglass)
domcharrier pushed a commit to ROCm/hipDF that referenced this pull request Dec 12, 2024
GALI PREM SAGAR
Thu Nov 16 16:22:44 2023 -0600
Raise error in `reindex` when `index` is not unique (#14400) (#14429)
Bacport of #14400 
Fixes: #14398 
This PR raises an error in `reindex` API when reindexing is performed on a non-unique index column.

Authors:
  - GALI PREM SAGAR (https://github.com/galipremsagar)

Approvers:
  - Matthew Roeschke (https://github.com/mroeschke)
  - Lawrence Mitchell (https://github.com/wence-)

URL: rapidsai/cudf#14400

Authors:
   - GALI PREM SAGAR (https://github.com/galipremsagar)

Approvers:
   - Richard (Rick) Zamora (https://github.com/rjzamora)
   - Ashwin Srinath (https://github.com/shwina)
   - Ray Douglass (https://github.com/raydouglass)
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 breaking Breaking change bug Something isn't working Python Affects Python cuDF API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Reindexing on duplicate values should error via constructors
4 participants