-
Notifications
You must be signed in to change notification settings - Fork 889
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 sort gtests coded in namespace cudf::test #12237
Fix sort gtests coded in namespace cudf::test #12237
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## branch-23.02 #12237 +/- ##
===============================================
Coverage ? 88.26%
===============================================
Files ? 137
Lines ? 22586
Branches ? 0
===============================================
Hits ? 19935
Misses ? 2651
Partials ? 0 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
What's the advantage to removing the default parameters to the segmented_sorted_order()
functions? It makes those declarations inconsistent with others in sorting.hpp
This helps with removing the default for the stream parameter. |
@gpucibot merge |
…ns (#12254) Removes the default parameter declarations from `cudf::detail::sort`, `cudf::detail::sorted_order`, `cudf::detail::sort_by_key`, `cudf::detail::stable_sorted_order`, and `cudf::detail::stable_sort_by_key` as well as `cudf::detail::quantile`, `cudf::detail::quantiles` and `cudf::detail::percentile_approx` Reference comment: #12237 (comment) Also fixes incorrect stream parameter passed within a `quantiles.cu` detail function. No function has been added, modified, or changed. Authors: - David Wendt (https://github.com/davidwendt) Approvers: - Mark Harris (https://github.com/harrism) - Nghia Truong (https://github.com/ttnghia) - Yunsong Wang (https://github.com/PointKernel) URL: #12254
Description
Fixes sort gtests source files coded in namespace cudf::test
The segmented_sort_tests.cpp also included calls to detail functions which were unnecessary. These were changed to call the external API instead. This also allowed removing the default parameters from the segmented-sort detail functions.
No function or test has changed just the source code reworked per namespaces.
Reference #11734
Checklist