fixes/improvements to benchmark page #19
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mildsunrise
commented
Aug 1, 2025
mildsunrise
commented
- Bunch of obvious fixes so that things work again
- Add more sizes
- Make benchmarks a bit more exact by having them only calculate FFTs, nothing else
- Add an FFT I wrote which is about ~50 lines of code and performs more or less like PulseFFT
- Some other small improvements
- don't compute total as part of the benchmarks pls - add 8192 size
- report FFT points, not total - pre-generate a few buffers - use random data - remove dead code
- extract two things to variables - use f32 (for fairness with pulsefft which also uses f32)
implement full Cooley-Tuckey (not possible with WebGL, which is what I wrote this code for)
Author
|
in case someone is interested, I made a more proper benchmark that adds more implementations (some are ~twice as fast as PulseFFT), performs the execution in a dedicated worker, and plots the results in a nice graph. However it only compares complex FFTs. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.