-
Notifications
You must be signed in to change notification settings - Fork 887
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
Fix scatter output size for structs. #8155
Conversation
1. Output struct size must match target column, not source. 2. Test case for rapidsai#1. 3. Also, move scatter_struct_tests and scatter_list_tests from .cu to .cpp, for faster compile.
I'd better let @harrism off the hook on this one. Thanks for the reviews, @ttnghia && @davidwendt. |
Codecov Report
@@ Coverage Diff @@
## branch-0.20 #8155 +/- ##
===============================================
+ Coverage 82.88% 82.92% +0.03%
===============================================
Files 103 104 +1
Lines 17668 17896 +228
===============================================
+ Hits 14645 14840 +195
- Misses 3023 3056 +33
Continue to review full report at Codecov.
|
@gpucibot merge |
Fixes #8150, which causes CUDF failures if the
scatter()
source struct column has a different size from that of the target. It turns out that the implementation was erroneously using the source column's size to construct the result, instead of the target's.The following changes were made in this commit:
scatter_lists_test
andscatter_struct_test
were moved from.cu
to.cpp
, for faster compile.