Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vectorized implementation #49

Open
MatthieuBizien opened this issue Feb 4, 2021 · 2 comments
Open

Vectorized implementation #49

MatthieuBizien opened this issue Feb 4, 2021 · 2 comments

Comments

@MatthieuBizien
Copy link

MatthieuBizien commented Feb 4, 2021

StackOverflow just reminded me of my old post, where I made your implementation of dtw 435 times faster. I have no use of it now (the Kaggle competition has ended years ago), but maybe others could use it.

The solution is very simple, just removing the for loops :) It would however needs some adaptation as your code have changed since.

@pierre-rouanet
Copy link
Member

Thanks, indeed this could really save some CPU time :)
I'l try to integrate it when I have some free time!

@junhyeokahn
Copy link

Could you elaborate a little more on this?
For instance, which part of the codes should I change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants