chaiNNer icon indicating copy to clipboard operation
chaiNNer copied to clipboard

Uninstalling chaiNNer does not remove appdata files

Open Kim2091 opened this issue 2 years ago • 3 comments

I recently uninstalled chaiNNer to diagnose some issues, and noticed that uninstalling it does not actually clear python, the dependencies, etc. stored in the AppData directory. I had to manually delete these to reinstall chaiNNer fully.

Kim2091 avatar Nov 28 '23 22:11 Kim2091

Oof, that's definitely an issue... I think i know what i can do to fix that. I think squirrel has hooks for where you can run things on uninstall -- so i might be able to get it to clear the python folder out when that happens. Amazingly, i don't think i've ever actually even tried uninstalling chaiNNer lol

joeyballentine avatar Nov 29 '23 03:11 joeyballentine

I recently uninstalled chaiNNer to diagnose some issues, and noticed that uninstalling it does not actually clear python, the dependencies, etc. stored in the AppData directory. I had to manually delete these to reinstall chaiNNer fully.

yes this is what I had to do as well to uninstall.

vxish avatar Nov 30 '23 13:11 vxish

On linux, the directory is $HOME/.config/chaiNNer It's north of 5Gb after uninstall ...

Another directory that is left over is $HOME/.cache/chainner_pip

ipeevski avatar Jan 10 '24 22:01 ipeevski

The patch doesn't seem to cater for linux installs?!

ipeevski avatar Apr 06 '24 03:04 ipeevski

I believe we only have the ability to hook into the windows installer. Nothing we can do for linux afaik

joeyballentine avatar Apr 06 '24 03:04 joeyballentine

What about just rm? The above two directories would be a great start.

ipeevski avatar Apr 06 '24 09:04 ipeevski

The original issue was only about windows. I made a new issue for Linux #2751.

What about just rm?

The problem isn't which command to use to remove files, but how we can run our code/commands during the uninstallation process. Electron's (un)installer for windows gives us an easy way to hook into the installation process, but we'll have to investigate how that works for Linux.

RunDevelopment avatar Apr 06 '24 10:04 RunDevelopment