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: add dropout check to fft plotter #58

Merged
merged 4 commits into from
Mar 26, 2017

Conversation

kd0aij
Copy link
Contributor

@kd0aij kd0aij commented Mar 23, 2017

we're regularly dropping every 10th accel sample. Need to check whether this is happening at the sensor, or just in the log:

sample range:  4382 38370
N samples:  33988
ndeltas:  33987
duration: 37.785 seconds
average sample rate: 899 Hz
mean sample interval:  1.11 msec
std:  342 usec

dropping_samples

@kd0aij
Copy link
Contributor Author

kd0aij commented Mar 23, 2017

resampling using numpy.interp, appears to have added harmonic distortion. This is likely due to using the SampleUS values as timestamps. Will have to preprocess them to remove the jitter before resampling.

resampled

@kd0aij kd0aij changed the title add dropout check to fft plotter WIP: add dropout check to fft plotter Mar 24, 2017
@kd0aij kd0aij force-pushed the add_dropout_check branch from 0b7fb03 to 0232ec8 Compare March 25, 2017 14:06
@kd0aij
Copy link
Contributor Author

kd0aij commented Mar 25, 2017

this version assumes the SampleUS fields in ACCn and GYRn log records are sequential sample counters representing the total number of samples received from the sensor in fast_sampling mode since the last fifo error or reset. It also computes the sample rate by comparing SampleUS to the timestamps (TimeUS)

@kd0aij
Copy link
Contributor Author

kd0aij commented Mar 26, 2017

the hardwired sample rates don't look right for fmu-v4

@tridge tridge merged commit 7f9ae37 into ArduPilot:master Mar 26, 2017
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