slaypni / fastdtw

A Python implementation of FastDTW
MIT License
794 stars 123 forks source link

fastdtw and dtw giving different answers. #20

Closed harsh244 closed 6 years ago

harsh244 commented 6 years ago

timeseries1.xlsx timeseries2.xlsx

i tried fastdtw and dtw on the two timeseries that I have uploaded.Both are giving different answers. Should it give the same answer or am I missing out something?

kernc commented 6 years ago

@harsh244 It's good practice to explain briefly why the issue is no longer relevant before closing it.

harsh244 commented 6 years ago

After reading the paper found out that it depends a lot on how we set the radius parameter so fasdtw may not always give us the best path as it is based on a lot of approximations ,the more we set the radius as the more closer the answer of our fasdtw to dtw ,at radius=(length of timeseries) ,fastdtw becomes dtw .