-
Notifications
You must be signed in to change notification settings - Fork 428
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
Speed comparison #19
Comments
Thanks for sharing the speed comparison demo @caub! I wasn't convinced by "way slower" so I've uncommented some code to find out that pvorb aka node-md5 is about 50 times slower (10MB test file, 3-yo MBP). I believe it could make many developers life easier if this package included a speed comparison table in the README file. |
Confirm. I think it's due to the using of |
I tested it on qemu ppc64 and it produced wrong results as expected. |
It's was used in a browser (Do they work fine in BE-machines?), or in Node.js? |
Which libs are broken on Big-Endian machines? |
added benchmarks to README |
just did this https://jsfiddle.net/crl/6uaktq3s/ that shows that your lib is the fastest around, and deserve more credit
The text was updated successfully, but these errors were encountered: