-
Notifications
You must be signed in to change notification settings - Fork 43
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 benchmarks for sparse unary minus #45
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 for this, after making the minor changes I suggested it should be good to go!
# unary operations # | ||
#################### | ||
|
||
g = addgroup!(SUITE, "unary ops") |
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.
Hmm, the linear algebra benchmarks have an "arithmetic"
group, can we replace "unary ops"
with "arithmetic"
to make things consistent?
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.
arithmetic
is much better. Thanks!
|
||
# unary minus, julialang repo issue #19503 / fix #19530 | ||
g["unary ops", "unary minus", size(small_sqr)] = @benchmarkable -$small_sqr | ||
g["unary ops", "unary minus", size(large_sqr)] = @benchmarkable -$large_sqr |
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.
The extra group key here isn't necessary, e.g. g["unary ops", "unary minus", size(small_sqr)]
can be changed to g["unary minus", size(small_sqr)]
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.
Fixed on push. Thanks!
Awesome, thanks again! I'll let you know once these benchmarks are available on Nanosoldier (they still need to undergo a |
Cheers, thanks |
This pull request adds benchmarks for sparse unary minus as discussed in JuliaLang/julia#19530. Apologies in advance for inevitable BaseBenchmarks.jl-newbie errors. Thanks and best!