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.

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

[HOTFIX] v25.02.01 #18112

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

[HOTFIX] v25.02.01 #18112

wants to merge 3 commits into from
+110 −95

Conversation

AyodeAwe
Copy link
Contributor

v25.02.01 cuDF hotfix release

Matt711 and others added 3 commits February 19, 2025 13:31
@Matt711
Skip the failing connectorx polars tests (#18037)
c99f393 
In #18015, we tried skipping the failing polars tests and applying the
workaround mentioned in polars issue 21274. But pip is [unable to solve
our test
environment](https://github.com/rapidsai/cudf/actions/runs/13406947992/job/37463788766)
in that case. This PR just skips the tests because we only need to do
one or the other, not both.
@nvdbaranec
Fix 'Unexpected short subpass' exception in parquet chunked reader. (#…
da5d0a1 
…18019)

Fixes  #18043

An incorrect computation in the subpass generation code would come to
the conclusion that there weren't enough rows to decode for list columns
under certain circumstances.

This PR fixes the issue and I did a little bit of variable naming
cleanup around the area. Ultimately the true source of the bug was
poorly named variables causing them to be used incorrectly.

Edit: I've disabled various checks in the chunked reader tests that
expect specific chunk counts being returned from chunking operations.
Changes to decompression temporary memory usage can make this
unreliable. We will need a smarter solution down the road.
@bdice
Update to nvcomp 4.2.0.11 (#18042)
1fe744f 
This updates to use nvcomp 4.2.0.11.

The version is updated in rapids-cmake in
rapidsai/rapids-cmake#780.
@AyodeAwe AyodeAwe requested review from a team as code owners February 26, 2025 21:11
@github-actions github-actions bot added libcudf Affects libcudf (C++/CUDA) code. Python Affects Python cuDF API. cudf.polars Issues specific to cudf.polars labels Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jameslamb jameslamb Awaiting requested review from jameslamb jameslamb is a code owner automatically assigned from rapidsai/packaging-codeowners

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

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

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

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

At least 3 approving reviews are required to merge this pull request.

Assignees

@AyodeAwe AyodeAwe

Labels
cudf.polars Issues specific to cudf.polars libcudf Affects libcudf (C++/CUDA) code. Python Affects Python cuDF API.
Projects
cuDF Python
Status: In Progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AyodeAwe @Matt711 @nvdbaranec @bdice