biotracker_core icon indicating copy to clipboard operation
biotracker_core copied to clipboard

Fresh tracks were unable to be saved

Open failure-to-thrive opened this issue 7 years ago • 1 comments

The deleted code induced #156. Of course, getMaxLinecount() returns only 1 when a tracking is not performed yet. I have no idea why that code was there, but it looks to be there intentionaly. So if I've missed something, just dismiss this PR.

failure-to-thrive avatar Aug 05 '18 13:08 failure-to-thrive

Well, yes, it was there intentionally. Technically a trajectory needs at least two points to be considdered a trajectory (per definition). Maybe there are usecases for writing a trajectory consisting of a single point, so replacing <=1 with <1 seems reasonable.
What is a valid usecase for writing empty trajectories? The downside of course would be, that every time some kind of media is opened, a "trajectory" would be written, although the tracking plugin was never triggered.

hmoenck avatar Aug 08 '18 17:08 hmoenck