-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fix rxFrequencies #107
Fix rxFrequencies #107
Conversation
yes, using |
The first test fail was Aqua finding am ambiguity between Base and Interpolations, I bumped the Aqua compat where the default behaviour is now to only look at the tested package. |
I think we also need to adapt the frequency calculations in the frequency filters to stay consistent. For example minFreq -> minFreqIdx should be the same as the one from rxFrequencies. To reuse |
Moved Bruker UUID Generation and UniformRandomPositions to StableRNG
Closes #105, maybe fixes #101
I changed some functions in Derived.jl to use FFTW or built-in julia functions and added tests, which would have caught issue #105