-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
mTECS branch wind estimation #1036
Conversation
@thomasgubler @julianoes Review would be appreciated. It would be good to have a larger number of logs to see if the wind speed estimation is reliable and can be used. Not high on the priority to fix it, but we need a lot of data, so this should go in soon-ish. |
Why are you defining wind speed only as a 2d velocity instead of 3d? |
We can't estimate it in 3D with a pitot facing only into the plane and movements that are almost exclusively in 2D. |
Well, not without a good model... |
Because we should extend interfaces as they become relevant and not create empty fields that we have no clue yet how to use. There is no reliable measurement in Z direction and you can go as far as to argue that its not observable without forcing the system into a very special motion pattern and/or add 3D airspeed sensing. Both is not relevant for at least months if not longer and adding fields is possible ad-hoc (the log format is self-describing). |
Tested in HIL, no analysis of correctness of estimate done, but no operational problems found, merging |
mTECS branch wind estimation
This adds logging of the wind estimate. Compiles, not tested in HIL / flight yet.