-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added missing function definitions resolves #64 #65
Conversation
I guess the failed checks were already there for Float16sr, right? |
Yes I believe there's something wrong with the subnormals (probably in general) but because there's so rarely used, it only shows up with Float16sr. I need to look into this. The probably at the moment is that it happens a (only very) few times that the rounding error is more than |
As resolved by you the following function definitions have been added to Float16sr and Float32sr for compatibility with GenericFFT:
I have tested the code. It works well with GenericFFT.
Also, I use mod function in PIC so I have added the definition for it under basic operations as well for both Float16sr and Float32sr.