d-ronin / dRonin

The dRonin flight controller software.
http://dronin.org
Other
289 stars 167 forks source link

Python API: unable to read new logs (error in telemetry.py) #2239

Open raemin opened 5 years ago

raemin commented 5 years ago

Issue details: Can log older logs (presumably wired), but Inconceivable logs crash the logviewer.

Telemetry.py , error line 1017, no header signature

Logs.zip (edit : discard these, as I made a "better" test later, see comments)

2019-01-11

raemin commented 5 years ago

I've reverted back to wired, made a clean install, formated the openlagger sdcard, generated the log and could read them with the "pip installed" logviewer. No crash, nice graphs.

From then on I upgraded the firmware to inconceivable (2126), made another log and attempted to read it with the python scripts hosted on github (branch next as of april 8th). The viewer does not crash anymore, but all the graphs are empty.

See corresponding sample logs and screenshots.

SampleLogs.zip

raemin commented 5 years ago

Commenting again this issue ... The problem seems to be related to the latest python source! In fact I was attempting to read wired logs with wired scripts and inconceivable logs with inconceivable scripts. It seems wired python scripts are capable to read both wired/inconceivable logs, but inconceivable scripts do only produce blank graphs.

If you take the samples provided in my previous post, all the logscan be viewed using the python scripts from the "wired" branch.

2019-04-10

there are a few warnings though

2019-04-10 (1)

Now the same "inconceivable" log as displayed by the "next" python scripts (as of April 8th)

2019-04-10 (2)

No warning in the console with these latest scripts.

tracernz commented 5 years ago

Thanks for the report, and your work to narrow it down. When time permits I can have a look at this.