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

Invalid location data was found in that log, ignoring #159

Open
hyperion11 opened this issue Sep 29, 2015 · 9 comments
Open

Invalid location data was found in that log, ignoring #159

hyperion11 opened this issue Sep 29, 2015 · 9 comments

Comments

@hyperion11
Copy link

Can't upload log.
Error: Invalid location data was found in that log, ignoring
Link to my log
https://www.dropbox.com/s/d9bqa3fnn1zau4z/560941b2a68980bb376ed700.log?dl=0

@rmackay9
Copy link

I've found the same issue with all Copter-3.3 logs

@mrpollo
Copy link
Member

mrpollo commented Oct 13, 2015

thanks @hyperion11, @rmackay9 this seems to be a different issue from what we have originally found for 3.3 (TimeMS vs TimeUS), I'm reviewing.

@hyperion11
Copy link
Author

@mrpollo, if manually edit log and delete all lines with "err" log successfully uploading. copter 3.2.1.
May be automatically reject lines with errors while uploading?

@peterbarker
Copy link

The log is malformed. Here's the format:

FMT, 19, 5, ERR, BB, Subsys,ECode

here's an example ERR line:

ERR, 16, 2

What's missing there is a sequence number:
CTUN, 542046, 640, 12, 652, 0, 0.38, 0.35, 0.00, 0.00, -250, 51
ATT, 542106, -17.94, -11.56, 0.00, 2.35, 30.24, 33.38, 0.19, 0.78

That second column is a sequence number - missing from ERR.

Where did this log come from?

@rmackay9
Copy link

In 3.2.1 the ERR message didn't have a time on the front perhaps. That shouldn't cause the whole upload to fail though I think.

@mpjf01
Copy link

mpjf01 commented Oct 16, 2015

I get the same error message and failed result with all log files
This is the shortest I have - change .gif to .bin
.85

@rmackay9
Copy link

@peterbarker has fixed the LogAnalyzer.py script which I think was the main blocker for DroneShare not working. I think @mrpollo needs to incorporate that script change into Droneshare now.

@mrpollo
Copy link
Member

mrpollo commented Oct 16, 2015

Sadly the LogAnalyzer.py isn't the only culprit here, we have some dependencies we need to fix in order to be able fix the main problem which seems to be some data flash log syntax changes, the code is tightly coupled to the messages old format.

project in question is here: https://github.com/dronekit/arduleader

@imrj
Copy link

imrj commented Dec 8, 2015

is this going to be fixed any time soon? its a shame the site has become unusable after AC release 3.3.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants