dronekit / droneshare

DroneShare is a shared repository of drone missions built using DroneKit-Cloud
http://www.droneshare.com/
Apache License 2.0
84 stars 66 forks source link

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

Open hyperion11 opened 9 years ago

hyperion11 commented 9 years ago

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 commented 9 years ago

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

mrpollo commented 9 years ago

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 commented 9 years ago

@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 commented 9 years ago

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 commented 9 years ago

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 commented 9 years ago

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 commented 9 years ago

@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 commented 9 years ago

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 commented 8 years ago

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