mcmilk / 7-Zip-zstd

7-Zip with support for Brotli, Fast-LZMA2, Lizard, LZ4, LZ5 and Zstandard
https://mcmilk.de/projects/7-Zip-zstd/
Other
5.01k stars 298 forks source link

docs (lzip): add lzip to benchmark and replace "LZ4, LZ45" in README.md by "lzip (LZ4, LZ5)" #13

Closed GitMensch closed 7 years ago

GitMensch commented 7 years ago

Quoting from https://sourceforge.net/p/sevenzip/discussion/45797/thread/32771d84/#e23d :

Hello,

I added the lzip patch to my modified 7-Zip version, which also has LZ4, LZ5 and Zstandard multithreaded compression / decompression in it...

And its in the docs, too. Shouldn't it be added to the README.md (and maybe the tags)?

mcmilk commented 7 years ago

The README.md has a note about it:

included lzip decompression support, patch from http://download.savannah.gnu.org/releases/lzip/7zip/

Just search for lzip here: https://github.com/mcmilk/7-Zip-zstd

GitMensch commented 7 years ago

Hm, ok, I see the note now. I think I was blinded by the headline teaser and the codec list naming LZ4 and LZ5 and the nice benchmarks only about zsdt and LZMA.

mcmilk commented 7 years ago

Yes, measuring the decompression speed with LZ4 or LZ5 isn't usefull with the 210MB Silesia compression corpus... I am searching for some inofficial bigger corpus.... so they are not included in the benchmark... currently.

If someone can help me with that, just do it ;-)

GitMensch commented 7 years ago

Can I suggest to change all occurrences of "LZ4, LZ45" to "lzip (LZ4, LZ5)? Or did I get something wrong here?

Yes, measuring the decompression speed with LZ4 or LZ5 isn't usefull with the 210MB Silesia compression corpus...

Why not?

mcmilk commented 7 years ago

It's simply to fast. The measuring time will be small to get valid results :-)

But I will create some own compression corpus for this and put it onto github...

mcmilk commented 7 years ago

Lzip is not supported by the "plugin only installation" ... only the full setup has support for it. So there it is in the documentation ... everything is okay with it.

GitMensch commented 7 years ago

Side question: What are the reasons for Lzip not being available with the plugin only installation?

mcmilk commented 7 years ago

It's not possible... that's the reason. I included the lzip patch from http://download.savannah.gnu.org/releases/lzip/7zip/ to the own full setup... nothing more.

The patch does not create plugins... ;)