UCBerkeleySETI / hyperseti

A SETI / technosignature search code to find intelligent life beyond Earth
https://hyperseti.readthedocs.io
10 stars 4 forks source link

Remove dependency on cusignal #71

Closed telegraphic closed 1 year ago

telegraphic commented 2 years ago

Currently cusignal is needed for argrelmax. cusignal is a pain to install and incompatible with pip, you need to use conda.

cusignal also does some weird precompile at installation time, so you can't just grab the code from the repo and throw it in to hyperseti codebase.

Best approach will be to carefully write our own peak finding algorithm on the GPU. This is currently the bottleneck of the whole pipeline so most fruitful area to optimize.

telegraphic commented 1 year ago

New peak finding algorithm in #83. Still has some issues though :/