aio-switch-updater icon indicating copy to clipboard operation
aio-switch-updater copied to clipboard

Very slow extract for large pack

Open lowburger opened this issue 3 years ago • 3 comments

Describe the bug Hi, I make my own package 200MB unzipped with atmosphere and many files homebrew and tools. Extract time almost 20 minutes! I test on NX-shell tool extracting and it takes 2 minutes extract, is it normal?

To Reproduce Only a large pack with thousands of file and 200MB before zip.

Screenshots/Crash code/Crash reports Nothing.

System Firmware Version and CFW Version

  • System Firmware: [12.1.0]
  • CFW Version [Atmospshère 0.19.5]

SD card format FAT32

Additional context Nothing.

lowburger avatar Aug 09 '21 09:08 lowburger

Well, each to-be-extracted file is checked for a number of conditions, so the more files the worst it is. I'm using a C++ library for all the extraction process, and I'm sure there is some overhead to that too, but I don't know if I'm willing to look into too much. I'll keep this in mind however

HamletDuFromage avatar Aug 09 '21 12:08 HamletDuFromage

It works great for something small but I made a bigger pack to share with some friends and had the same issue, would be great if you could improve the speeds.

pixeltester avatar Sep 25 '21 16:09 pixeltester

@HamletDuFromage Ran into this myself and it does seem needlessly slow if you package a lot of files. Perhaps a solution would be to allow users to whitelist directories to skip your checks?

ghost avatar Nov 04 '21 16:11 ghost

fixed in https://github.com/HamletDuFromage/aio-switch-updater/commit/64c13a5626b3645d7f14942d97b1f291a9aae540

HamletDuFromage avatar Aug 07 '22 13:08 HamletDuFromage

fixed in 64c13a5

Feature ready or needs more develop? I test and works fast but crash on 95% extract all attempts.

lowburger avatar Aug 08 '22 17:08 lowburger

The extraction crashes if you include aio-switch-updater.nro in your zip, so that might need a self exception.

ghost avatar Aug 09 '22 03:08 ghost

This fix the crash, thx

lowburger avatar Aug 09 '22 20:08 lowburger