containers icon indicating copy to clipboard operation
containers copied to clipboard

FIX fast stable diffusion pod template TMALLOC ( memory problem fix )

Open nerdylive123 opened this issue 10 months ago • 4 comments

like on https://github.com/AUTOMATIC1111/stable-diffusion-webui/discussions/6722#discussioncomment-5609254. This is a fix for memory problem when user switched or loaded models/lora multiple time, then the SD will crash after.

pls do tests on this fix, i don't use the minimal version, i usually use the normal version of libtcmalloc.

nerdylive123 avatar Apr 16 '24 18:04 nerdylive123

like on AUTOMATIC1111/stable-diffusion-webui#6722 (comment). This is a fix for memory problem when user switched or loaded models/lora multiple time, then the SD will crash after.

pls do tests on this fix, i don't use the minimal version, i usually use the normal version of libtcmalloc.

Minimal version works fine, but I wonder why you're installing the deb packages from TheLastBen's Github repo rather than simply installing the Ubuntu apt packages.

ashleykleynhans avatar Apr 16 '24 18:04 ashleykleynhans

Well yeah, idk i think it would be great to use the deps from that repo

nerdylive123 avatar Apr 16 '24 19:04 nerdylive123

Well yeah, idk i think it would be great to use the deps from that repo

I would agree with you if they were related to the repo itself, but its pretty bad practice to use debian packages that are version locked and very likely outdated. Its best practice to install the latest packages from the Operating System repositories (in this case the Ubuntu apt repositories) to ensure that you get the latest security patches included.

ashleykleynhans avatar Apr 16 '24 19:04 ashleykleynhans

@nerdylive123 @ashleykleynhans When can we get this merged?

ercanozer avatar May 04 '24 15:05 ercanozer