-
Notifications
You must be signed in to change notification settings - Fork 902
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
Add benchmark for strings/fixed_point convert APIs #8417
Add benchmark for strings/fixed_point convert APIs #8417
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.
lgtm, other than the two comments (maybe it isn't possible)
Codecov Report
@@ Coverage Diff @@
## branch-21.08 #8417 +/- ##
===============================================
Coverage ? 82.83%
===============================================
Files ? 109
Lines ? 17901
Branches ? 0
===============================================
Hits ? 14828
Misses ? 3073
Partials ? 0 Continue to review full report at Codecov.
|
@gpucibot merge |
Reference #5698
This PR adds benchmark tests for
cudf::strings::to_fixed_point
andcudf::strings::from_fixed_point
.The benchmarks include measurements for both
decimal32
anddecimal64
types.