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

Report error while set Zstandard compression level greater than 20 #109

Closed
OMICRON3069 opened this issue Dec 21, 2019 · 3 comments
Closed

Comments

@OMICRON3069
Copy link

Program report unspecified error while using zstd ultra mode:

F:\Z-Stack>7z a archiv.7z -m0=zstd -mx22  "[Cornflower.Studio] #03 ME!ME!ME! (BDRip 1080P X265 Main10p Flac)_2.mkv"

7-Zip 19.00 ZS v1.4.4 R1 (x64) : Copyright (c) 1999-2019 Igor Pavlov, 2016-2019 Tino Reichardt : 2019-11-13

Scanning the drive:
1 file, 392802336 bytes (375 MiB)

Creating archive: archiv.7z

Add new data to archive: 1 file, 392802336 bytes (375 MiB)



System ERROR:
Unspecified error

And if I test it with level 19, it's fine.

F:\Z-Stack>7z a archiv.7z -m0=zstd -mx19  "[Cornflower.Studio] #03 ME!ME!ME! (BDRip 1080P X265 Main10p Flac)_2.mkv"

7-Zip 19.00 ZS v1.4.4 R1 (x64) : Copyright (c) 1999-2019 Igor Pavlov, 2016-2019 Tino Reichardt : 2019-11-13

Scanning the drive:
1 file, 392802336 bytes (375 MiB)

Creating archive: archiv.7z

Add new data to archive: 1 file, 392802336 bytes (375 MiB)


Files read from disk: 1
Archive size: 392781638 bytes (375 MiB)
Everything is Ok

Windows 10 OS Build 18363.535
Direct install the exe file from 7-Zip ZS Releases

@mcmilk
Copy link
Owner

mcmilk commented Dec 21, 2019

I think it's an out of memory error, you can fix this by setting the number of threads to some lower value.

There is currently no nice way to give the user a nice message box with the real error ... I am out of time currently for looking into that a bit closer - sorry :/

@OMICRON3069
Copy link
Author

Test it again with 1 thread set in gui and also with a smaller file, but it will fail too.
Snipaste_2019-12-22_16-38-17
Snipaste_2019-12-22_16-39-07

And I start it will at least 3G available memory.
Snipaste_2019-12-22_16-38-39

I think it's totally OK that you can't have much time to do this. At least you have already bring some useable and awesome code to us. I'm just pointing an issue. Stick with level 19 seems enough for me.

@mcmilk
Copy link
Owner

mcmilk commented Apr 14, 2020

I checked it and I can catch the out of memory message ... I just added this now, next release will give you the correct failure I think.

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

No branches or pull requests

2 participants