-
Notifications
You must be signed in to change notification settings - Fork 27
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
Levenstein distance performance #33
Comments
Hello @marcomorain, I've just run your test and must say you are right. There is something fishy here.I start investigating. |
thanks! On Wed, Jan 7, 2015 at 6:35 PM, Guillaume Plique notifications@github.com
|
I just deployed a faster version thanks to this gist's implementation without relying on external Can you test that and tell me if this feels better? |
I've just changed my code to use this impl: http://en.wikibooks.org/wiki/Algorithm_Implementation/Strings/Levenshtein_distance#Clojure I'll try your again if I get some free time tomorrow. Thanks! On Wed, Jan 7, 2015 at 7:02 PM, Guillaume Plique notifications@github.com
|
⭐
Awesome! |
🚀 |
The
levenshtein/distance
function has very poor performance for even short strings – is this a known issue? It takes 10 seconds on a MacBook Pro (3 GHz Intel Core i7) running Java 8.The text was updated successfully, but these errors were encountered: