-
Notifications
You must be signed in to change notification settings - Fork 231
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
feat: add FFTBatch
benchmark
#515
Conversation
FFTBatch
benchmarkFFTBatch
benchmark
docs(bench): introduce FFTBatch benchmark should be typed with test |
9cdd601
to
0e17e9d
Compare
c7eec5e
to
2b869e9
Compare
b9be430
to
2d3611e
Compare
2d3611e
to
3c047e3
Compare
I think
|
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
c116f77
to
47856fe
Compare
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
0f95a6f
to
41510b7
Compare
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.
94bae27
folders -> folder
dd033f2
to
9de7ad7
Compare
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
7f90689
to
81cff39
Compare
Previously, we had different implementations in each benchmark folder. With this commit, we remove it and make 1 generic reporter.
81cff39
to
ab9b4af
Compare
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
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
Description
This PR adds
FFTBatch
benchmark of Tachyon and Plonky3.