wannesm / dtaidistance

Time series distances: Dynamic Time Warping (fast DTW implementation in C)
Other
1.08k stars 184 forks source link

Support the same parameters in all DTW functions (#115) #150

Closed probberechts closed 2 years ago

probberechts commented 2 years ago

An attempt to unify the API of all DTW functions (as discussed in #115).

I am not sure what to do with the only_ub parameter. Currently, it is only supported by the dtw.distance and dtw.distance_fast parameters. To me it would make most sense to remove it from these functions, since the same functionality is already implemented in the ed module.