-
Notifications
You must be signed in to change notification settings - Fork 157
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
Use grid_stride_range in kernel loops #1178
Use grid_stride_range in kernel loops #1178
Conversation
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.
Thanks!
Do we want to move ranger into the rapidsai org or somewhere else shared before we introduce a dependency on it here? |
I am not opposed to that. That would make it easier for e.g. cuDF to adopt it. |
Decision is to merge this as-is for now. We can look at re-homing ranger if and when it is determined to be needed. |
/merge |
Closes #650
Description
Adds a dependency on Ranger and uses
ranger::grid_stride_loop
with a range-based for loop in outer loops in every raw kernel in cuSpatial. In the few cases where we had raw kernels without grid-stride loops, this adds one. Also adds guidance to the C++ developer guide.Checklist