gpsbabel icon indicating copy to clipboard operation
gpsbabel copied to clipboard

Interpolation debug left turned on

Open robertlipe opened this issue 2 months ago • 1 comments

./gpsbabel -i gpx -f bigw.gpx -x kalman -o gpx -F bigw-filered.gpx kalman: [GEN] interpolated point at "Mon Sep 22 14:39:57 2025 GMT" "interpolated2-1" lat: XX.XXXX lon: -YY.YYYY

This is disturbing in two ways.

A) That it's turned on and I didn't notice it. B) That I didn't notice it. This must mean the test suite never hits it, which is weird because there were definitely some spots that should be interpolated. (The sharp bend exiting the parking lot, turning south, just as the ride ends.)

robertlipe avatar Oct 16 '25 05:10 robertlipe

I don't think this else clause can ever be executed (besides being empty). https://github.com/GPSBabel/gpsbabel/blob/2356727fa40f29fc1e66b6db658ef776bc62cdbe/kalman.cc#L370-L373 The existence of the comment leads me to believe you expected a wpt could be marked for deletion but not be a zinger.

is_zinger_deletion and wpt_flags.marked_for_deletion are always set to true in adjacent lines.

tsteven4 avatar Oct 16 '25 16:10 tsteven4