Log download does not work

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Fly
2. Download log with APM Planner or use Terminal

What is the expected output? What do you see instead?
You cannot get log. 

What version of the product are you using? On what operating system?
APM 2.12, latest APM Planner 0.4.36. Same issue was in some earlier versions.

Please provide any additional information below.

Here is the APM Planner screenshot. First program says, that there is one log 
available. Few lines later program says, that there is no logs available.(There 
should be 3 or more flights in memory, so there should be 3 logs.)

It seems that there is something wrong with the last_log_num or get_num_logs. 
In the beginning the value is 1, but when program calls it again, result is 0.

----------------------------------------------
Free RAM: 2429
load_all took 3912us
using 2210 bytes of memory

Entering interactive setup mode...

If using the Arduino Serial Monitor, ensure Line Ending is set to Carriage 
Return.
Type 'help' to list commands, 'exit' to leave a submenu.
Visit the 'setup' menu for first-time configuration.

Move the slide switch and reset to FLY.

ArduPilotMega] logs
logs enabled:  ATTITUDE_MED GPS PM MODE CMD

1 logs available for download
Log number 1,    start page 2,   end page 390

Log] logs
logs enabled:  ATTITUDE_MED GPS PM MODE CMD

No logs available for download
Log] 
------------------------------------------
Here I add some line, now last_log is 0

Log] dump 1

Argc 2,    Dump_log 1,   Last Log 0
bad log number
logs enabled:  ATTITUDE_MED GPS PM MODE CMD

1 logs available for download
Log number 1,    start page 2,   end page 390

Original issue reported on code.google.com by alpo.has...@gmail.com on 26 May 2011 at 6:06

[GoogleCodeExporter]

It seems that log_step variable must be intialized:

byte get_num_logs(void)
{
    int page = 1;
    byte data;
    byte log_step = 0; // <------------------------ Add here this = 0

    DataFlash.StartRead(1);

Original comment by alpo.has...@gmail.com on 29 May 2011 at 6:00

[GoogleCodeExporter]

And APM Planner does not show log. Try Browse log, and you get very long error 
message. It does not show also old logs, that it show earlier.

Original comment by alpo.has...@gmail.com on 30 May 2011 at 12:18

[GoogleCodeExporter]

please try the latest planner on the svn

Original comment by Meee...@gmail.com on 1 Jun 2011 at 4:23

[GoogleCodeExporter]

Hi guys! Sometimes I dont get any logs available! I erased all the old logs, 
did some ground tests, with some power cycles, and some flights! Then I tried 
to download them and there are none available! 

Any ideas why this happens? 

Original comment by silvan...@gmail.com on 2 Jun 2011 at 4:23

[GoogleCodeExporter]

If I make modification mentioned in Comment 1 to APM, I can download logs with 
APM Planner 0.4.38. It writes kmz file, and I can look route in Google Earth.

But still APM Planner 0.4.38 does not show these logs in Log Browse, it gives 
very long error message. It can not also show earlier logs. 

So Log Browse does not work in APM Planner, and add the line mentioned in 
Comment 1 to APM code.

Original comment by alpo.has...@gmail.com on 2 Jun 2011 at 7:33

[GoogleCodeExporter]

alpo, can you attach the problem log to here?

Original comment by Meee...@gmail.com on 6 Jun 2011 at 12:59

[GoogleCodeExporter]

I have made the correction to APM code, so I can download logs with APM Planner.

Here is the log from this evening. Please note the following:

- In Google Earth planned route is not correct: Home position is marked 155 m 
from ground, it should be 0 m. Other points are correct, about 60 m from ground.

- In Google Earth there is some point with 0 as longitude. It is somewhere in 
Atlantic, other points are in Eastern Finland. Very often there are points 
somewhere in Africa, latitude is 0. These zero coordinates should be removed, 
when you make kml-file.

(- Nav pid in my plane is not yet correct :-)

Original comment by alpo.has...@gmail.com on 8 Jun 2011 at 6:47

Attachments:

• [08-06-11 08-35 10.zip](https://storage.googleapis.com/google-code-attachments/ardupilot-mega/issue-352/comment-7/08-06-11 08-35 10.zip)

[GoogleCodeExporter]

Original comment by Meee...@gmail.com on 16 Jun 2011 at 3:40

• Changed state: Accepted

[GoogleCodeExporter]

Is this issue still open?

Original comment by dewei...@gmail.com on 2 Jul 2011 at 9:31

[GoogleCodeExporter]

In download section last APM version is 2.1.2. Problem is still in this version.

In the forum there has been several questions abouth this log problem, for 
example this: http://diydrones.com/forum/topics/impossible-crash-due-to-apm-no

Original comment by alpo.has...@gmail.com on 3 Jul 2011 at 6:37

[GoogleCodeExporter]

Current firmware version is 2.2.1.  Please update your firmware.  I am not 
having any log issues with the current firmware, so would like to know if 
others do.

Original comment by dewei...@gmail.com on 5 Jul 2011 at 7:49

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

I had an identical problem (variable not automatically initialized to zero) 
with my own outopilot arduino based, it was in another tab and maybe it was an 
out of memory problem, only thing I know is that when I removed a certain 
function the variable didnt need to be manually intialized. At the time I wasnt 
able to check the memmory in the proper way so it could be that. I have log 
issue with apm2560 so probably is not a memory problem but didnt check.

Original comment by ingop...@tiscali.it on 6 Jul 2011 at 1:49

[GoogleCodeExporter]

No response for over two months, assuming fixed.

Original comment by amilcar....@gmail.com on 25 Sep 2011 at 9:19

• Changed state: Fixed