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

WIP: dynamic baro altitude + airspeed #20

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Stifael
Copy link
Contributor

@Stifael Stifael commented Feb 13, 2017

The file logdynamicpressure contains function for estimating airspeed from tilt angle (from paul priseborough) and functions to prepare data for dynamic barometric altitude change analysis.
The notebook "Log based dynamic barometer altitude" contains a first example of dynamic barometric altitude change estimation.
I added the two files here as a starting point for having a common px4 analysis repository, if that is something that is desired to have in the near future.
ToDo (in case the plan is to use px4tools as the common analysis tool):

  • add unit tests
  • improve the estimation of the static altitude / pressure instead of just taking the average for each altitude setpoint
  • change dynamic altitude estimation to dynamic pressure estimation. this requires to log baro pressure data

@coveralls
Copy link

coveralls commented Feb 13, 2017

Coverage Status

Coverage decreased (-0.5%) to 30.563% when pulling 12c95c1 on Stifael:airspeed_analysis_2 into 83e32e6 on dronecrew:master.

@@ -353,6 +353,21 @@ def series_quatrot(x, y, z, q0, q1, q2, q3, rot_name):
def series_quatrot_inverse(x, y, z, q0, q1, q2, q3, rot_name):
return series_quatrot(x, y, z, q0, -q1, -q2, -q3, rot_name)

def series_axangle2quat(x, y, z, yaw, msg_name):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is missing an unit test

@coveralls
Copy link

coveralls commented Feb 16, 2017

Coverage Status

Coverage decreased (-0.5%) to 30.563% when pulling b929837 on Stifael:airspeed_analysis_2 into 83e32e6 on dronecrew:master.

@coveralls
Copy link

coveralls commented Feb 21, 2017

Coverage Status

Coverage decreased (-0.5%) to 30.543% when pulling a7e9f23 on Stifael:airspeed_analysis_2 into 83e32e6 on dronecrew:master.

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.

3 participants