search

ethz-asl / fw_px4_plottools

Plotting tools for reading and showing Pixhawk uLog logs in Matlab
32 stars 22 forks source link

paramvector only capturing first value #34

Closed tstastny closed 3 years ago

tstastny commented 5 years ago

I dont know why this now suddenly stopped working .. or how it ever worked .. these things were tested before we merged. But it appears to be a bug when I run through it now. https://github.com/ethz-asl/fw_px4_plottools/blob/050e034fa801aed0a1c4857d4d64fe854115ee31/02_helper_functions/ImportPX4LogData.m#L195-L197

The snippet above can be fixed with the following replacement lines:

temp_params = temp_params(:,~isnan(temp_params(1,:)));
paramvector.(param_fields{idx_params}) = ...
   timeseries(temp_params(1,:)', temp_params(2,:)');
tstastny commented 5 years ago

Also need to convert timestamp to seconds

acfloria commented 3 years ago

Fixed by #37