Skip to content
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

Merged
merged 6 commits into from
Jun 2, 2014
Merged

mTECS branch wind estimation #1036

merged 6 commits into from
Jun 2, 2014

Conversation

LorenzMeier
Copy link
Member

This adds logging of the wind estimate. Compiles, not tested in HIL / flight yet.

@LorenzMeier
Copy link
Member Author

@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.

@thomasgubler
Copy link
Contributor

Why are you defining wind speed only as a 2d velocity instead of 3d?

@LorenzMeier
Copy link
Member Author

We can't estimate it in 3D with a pitot facing only into the plane and movements that are almost exclusively in 2D.

@thomasgubler
Copy link
Contributor

Well, not without a good model...
Anyway, but why not define the communication interface (uorb) in a more generic way?

@LorenzMeier
Copy link
Member Author

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).

@thomasgubler
Copy link
Contributor

Tested in HIL, no analysis of correctness of estimate done, but no operational problems found, merging

thomasgubler added a commit that referenced this pull request Jun 2, 2014
@thomasgubler thomasgubler merged commit b40fcb0 into mtecs Jun 2, 2014
@thomasgubler thomasgubler deleted the mtecs_wind_estimate branch June 2, 2014 20:25
@thomasgubler thomasgubler mentioned this pull request Jun 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants