search

UOA-PARC-SKA / FDAS

Fourier Domain Acceleration Search, FPGA-accelerated with OpenCL.
GNU General Public License v3.0
3 stars 0 forks source link

Harmonic summing candidate collection should use ring buffer #4

Closed jopperm closed 4 years ago

jopperm commented 4 years ago

The insertion index into the candidate buffers currently saturates on the last element:

https://github.com/ParallelAndReconfigurableComputing/skapint/blob/487d2c99f9bf2f8c89c8f85d8f63fffe15784b2b/FDAS/fdas/device/fdas.cl#L507-L509

Instead, a ring-buffer approach seems more appropriate here.

jopperm commented 4 years ago

This has been addressed in the rewrite of the harmonic summing code as of 9ab9c35872209c77201d652636c85a374c858bf2.