-
Notifications
You must be signed in to change notification settings - Fork 907
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
Implement reverse in libcudf #8410
Implement reverse in libcudf #8410
Conversation
To pass the style failure, you just need to add reverse.hpp to the file
|
Codecov Report
@@ Coverage Diff @@
## branch-21.08 #8410 +/- ##
===============================================
Coverage ? 82.94%
===============================================
Files ? 110
Lines ? 18186
Branches ? 0
===============================================
Hits ? 15084
Misses ? 3102
Partials ? 0 Continue to review full report at Codecov.
|
This looks good to me. I thought that we might want a gather map version of this since we seem to be moving towards gather maps, but the map is so trivial I don't see the value for it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once the naming thing is sorted I think this is fine to go.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, just noticed some test loops and vector<bool>
that can be cleaned up. Regarding "no raw loops", recommend this talk: https://www.youtube.com/watch?v=qH6sSOr-yk8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests look great!
rerun tests |
1 similar comment
rerun tests |
@gpucibot merge |
1 similar comment
@gpucibot merge |
Closes #7967