-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add lzwCompress.js and lz-string to the benchmarks #4
Comments
Good suggestion, you can add tests (benchmark) for these library following the tests in compress-json/test/compress-test.ts Lines 88 to 113 in 0566581
|
lz-string seems to be binary compression, may not be fair comparison (because the underline storage engine may need to escape binary string before storing them which can add the storage size significantly); I do not have access to the original sample data at the moment. Just did a trial with lzwcompress on another 50k sample data from the same online forum dataset. Number of sample object: 50,000
Remark: exact match consider whevether the order of object key is preserved when decompressed from the compressed representation. |
In fact, the one I was the most interested in is lz-string. |
It would be nice to be able to compare your library to those:
https://github.com/floydpink/lzwCompress.js
https://github.com/pieroxy/lz-string
Would it be possible to add them to the benchmarks ?
Thanks
The text was updated successfully, but these errors were encountered: