-
Notifications
You must be signed in to change notification settings - Fork 46
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
replaced look-up tables with <math.h> lib functions and updated press… #437
base: main
Are you sure you want to change the base?
Conversation
…ure altitude formula
This update requires changes to sensors.h/cpp from ptt/barocal and ptt/pitotspeed pull requests. |
I'm happy to see this being replaced, even though it's like saying goodbye to an old friend :). The STM32F103 didn't have a floating-point unit, so this was really the only way to get our loop time down under 1ms. I'm curious what the new timings look like with the H7. |
Yup, sad to see old friends retire. estimator_.run() went from 26us to 16us with the change. We still need to verify that there were no errors in the conversion. |
@avtoku I will put this on a board and do a test flight. I will let you know how it goes! |
Probably run it in the simulator first? |
Should we merge this? @iandareid |
Turbomath update