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

Incorrect handling of start-of-file noise #5

Closed
sco1 opened this issue May 20, 2022 · 0 comments
Closed

Incorrect handling of start-of-file noise #5

sco1 opened this issue May 20, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@sco1
Copy link
Owner

sco1 commented May 20, 2022

An attempt was made in b20f785 to mitigate noise at the beginning of the file causing an incorrect flight start by just trimming some data from the beginning of the file. Unfortunately, this didn't quite fix the problem, at least with the current default:

22-04-13_10-52-00

While the assumption that the FlySight will be on the ground when it's turned on is correct, we need a better way to discard segments that start very near to the beginning of the file. This may be better served with a longer trim default as a quick fix? It's also worth looking at the satellites in view to see if this is just from a lack of accuracy on startup.

I also think this threshold would be better in the flight segment classifier rather than just trimming off the beginning of the data file.

@sco1 sco1 added the bug Something isn't working label May 20, 2022
@sco1 sco1 closed this as completed in bd5e228 May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant