search

clMathLibraries / clFFT

a software library containing FFT functions written in OpenCL
Apache License 2.0
619 stars 192 forks source link

Support for radix 17 and 19 #216

Closed pieterhijma closed 6 years ago

pieterhijma commented 6 years ago

Dear all,

This pull requests adds support for radix 17 and 19 sizes. The formatting of the code is kept the same as the original files. Test cases have been revised to support these radices. I've tested the contribution but since not all tests pass in the develop branch, I compared test cases between the develop branch and the radix_17_19 branch. The results of these tests are in a separate git repository here, so the results can be reviewed or even be redone in a similar way. Finally, I also created a report with the findings on these tests that can be found here.

Please let me know if you need more information.

Kind regards,

Pieter Hijma

bragadeesh commented 6 years ago

@pieterhijma Thank you for your contribution, this looks great, let me do some testing on my side and get back to you

pieterhijma commented 6 years ago

@bragadeesh Great, thank you.

bragadeesh commented 6 years ago

Did basic testing, looks good, thanks

pieterhijma commented 6 years ago

Great, thanks.