search

garrettj403 / CZT

Chirp Z-Transform
MIT License
40 stars 14 forks source link

Create benchmark to find fastest t_method #9

Closed garrettj403 closed 3 years ago

garrettj403 commented 3 years ago

czt has several different options for t_method. These methods should be compared via a benchmark to find the fastest option, which should then become the default.

LarsHuebner-LHNW commented 3 years ago

There you go. method="mm" raises a MemoryError if array-size is too large, so don't get confused by the drop in computation time there. I did not test the f_method="fast" because of all the warnings about not working correctly. czt

garrettj403 commented 3 years ago

Thanks @LarsHuebner-LHNW

I didn't see your comment in time, so I created my own simple benchmark (see 1066414). I also found that t_method='scipy' is the fastest option.

I changed the default t_method to 'scipy' in 5009cdb.