edzer / trajectories

Handling and analyzing trajectory and movement data
31 stars 15 forks source link

dists #14

Closed skrackow closed 5 years ago

skrackow commented 6 years ago

It appears to me that dists method exhibits some inconsistencies/inconveniences.

  1. frechetDist might be given the tracks from compare function, not the original tracks, or some other method might hinder trajectories not overlapping in time to be compared.
  2. The indices of the dists matrix are interchanged. Hence, one gets out of bounds error with two Tracks lists of different lengths.
  3. For tracks not overlapping in time at all, compare might better not be called, so as to avoid the error message from there. Result will give NA and is therefore clearly understood, I'd guess. Hope those suggestions make some sense?! Thanks, SVEN.
skrackow commented 5 years ago

Found dists updated Dec 18, thanks very much!