Skip to content

Update deselected polars tests - #22265

Merged
rapids-bot[bot] merged 2 commits into
NVIDIA:mainfrom
wence-:wence/fix/skip-polars-import-test
Apr 23, 2026
Merged

Update deselected polars tests#22265
rapids-bot[bot] merged 2 commits into
NVIDIA:mainfrom
wence-:wence/fix/skip-polars-import-test

Conversation

@wence-

@wence- wence- commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

Description

Recent Polars changes have moved locations of some of them, or removed them, or a bugfix we were working around has gone in.

Checklist

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

@wence-
wence- requested a review from a team as a code owner April 23, 2026 11:26
@wence-
wence- requested a review from bdice April 23, 2026 11:26
@wence- wence- added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Apr 23, 2026
@Matt711

Matt711 commented Apr 23, 2026

Copy link
Copy Markdown
Member

I think one of the tests hit a timeout

@wence-

wence- commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author

I bumped the timeout. The H100 runners are all glacially slow

@wence-
wence- requested a review from a team as a code owner April 23, 2026 15:45
@wence-

wence- commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author

Now including #22189

wence- added 2 commits April 23, 2026 10:46
Recent Polars changes have moved locations of some of them, or removed
them, or a bugfix we were working around has gone in.
@vyasr
vyasr force-pushed the wence/fix/skip-polars-import-test branch from 91cd69e to 535a9d3 Compare April 23, 2026 17:46
@vyasr vyasr changed the title Update deselected polars tests Update deselected polars tests Apr 23, 2026
@vyasr vyasr changed the title Update deselected polars tests Update deselected polars tests Apr 23, 2026
@vyasr

vyasr commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

/merge

@rapids-bot
rapids-bot Bot merged commit 2998d25 into NVIDIA:main Apr 23, 2026
140 of 141 checks passed
shrshi pushed a commit to shrshi/cudf that referenced this pull request May 12, 2026
Recent Polars changes have moved locations of some of them, or removed them, or a bugfix we were working around has gone in.

Authors:
  - Lawrence Mitchell (https://github.com/wence-)

Approvers:
  - Matthew Murray (https://github.com/Matt711)
  - Peter Andreas Entschev (https://github.com/pentschev)
  - Vyas Ramasubramani (https://github.com/vyasr)

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

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants