search

jonkhler / s2cnn

Spherical CNNs
MIT License
939 stars 176 forks source link

Change from powf to sqrtf #46

Closed georg-bn closed 3 years ago

georg-bn commented 3 years ago

See Issue #41 . The FFT coefficients of certain frequencies were inaccurate due to numeric/rounding errors in powf(s, 0.5).

georg-bn commented 3 years ago

Here are the ints up to 2^14 where powf and sqrt differ on my setup with CUDA 10.1 difference_powf_sqrtf.txt.