Skip to content

Remove pyarrow from cudf_polars tests #19219

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 26, 2025
Merged

Remove pyarrow from cudf_polars tests #19219

merged 2 commits into from
Jun 26, 2025

Conversation

mroeschke
Copy link
Contributor

Description

With #19193 and this PR, we'll not import pyarrow explicitly in cudf_polars xref #18534

Checklist

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

@mroeschke mroeschke self-assigned this Jun 25, 2025
@mroeschke mroeschke requested a review from a team as a code owner June 25, 2025 20:31
@mroeschke mroeschke added Python Affects Python cuDF API. improvement Improvement / enhancement to an existing function labels Jun 25, 2025
@mroeschke mroeschke added the non-breaking Non-breaking change label Jun 25, 2025
@mroeschke mroeschke changed the title Remove pyarrow from tests Remove pyarrow from cudf_polars tests Jun 25, 2025
@github-actions github-actions bot added the cudf-polars Issues specific to cudf-polars label Jun 25, 2025
@GPUtester GPUtester moved this to In Progress in cuDF Python Jun 25, 2025
@mroeschke mroeschke mentioned this pull request Jun 25, 2025
Merged
3 tasks
@mroeschke
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 1ccf6dc into rapidsai:branch-25.08 Jun 26, 2025
91 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in cuDF Python Jun 26, 2025
@mroeschke mroeschke deleted the ref/cudf_polars/pyarrow_tests branch June 26, 2025 00:32
rapids-bot bot pushed a commit that referenced this pull request Jun 26, 2025
@mroeschke
Remove NumPy usage in cudf_polars (#19222)
b09a7c7 
Along with #19219, this PR removes our usages of NumPy in cudf_polars which is used for testing and multi-gpu utils. Since Polars does not depend on NumPy, neither should cudf_polars

Authors:
  - Matthew Roeschke (https://github.com/mroeschke)

Approvers:
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: #19222
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Matt711 Matt711 Matt711 approved these changes

@rjzamora rjzamora Awaiting requested review from rjzamora rjzamora is a code owner automatically assigned from rapidsai/cudf-python-codeowners

@brandon-b-miller brandon-b-miller Awaiting requested review from brandon-b-miller brandon-b-miller is a code owner automatically assigned from rapidsai/cudf-python-codeowners

Assignees

@mroeschke mroeschke

Labels
cudf-polars Issues specific to cudf-polars improvement Improvement / enhancement to an existing function non-breaking Non-breaking change Python Affects Python cuDF API.
Projects
cuDF Python
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mroeschke @Matt711