-
Notifications
You must be signed in to change notification settings - Fork 904
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
[REVIEW] Fix usecols
parameter handling in dask_cudf.read_csv
#9618
Conversation
Codecov Report
@@ Coverage Diff @@
## branch-21.12 #9618 +/- ##
================================================
- Coverage 10.79% 10.69% -0.10%
================================================
Files 116 117 +1
Lines 18869 19353 +484
================================================
+ Hits 2036 2070 +34
- Misses 16833 17283 +450
Continue to review full report at Codecov.
|
rerun tests |
@gpucibot merge |
Fixes: #9387
This PR fixes
usecols
parameter usage indask_cudf.read_csv
. When the csv read using byterange's the csv reader has to be passed complete column names innames
param but should passusecols
to return the exact columns that are needed only.