recognize icon indicating copy to clipboard operation
recognize copied to clipboard

Exclude models from updater backup

Open jplitza opened this issue 2 years ago • 3 comments
trafficstars

Describe the feature you'd like to request

Updating my Nextcloud via webinterface is no longer possible, since the updater tries for ages to backup the models under apps/recognize/models (as well as /apps/recognize/node_modules/@tensorflow/tfjs-node-gpu/deps/lib tensorflow.so – three times thanks to not supporting symlinks).

Describe the solution you'd like

Download the models to data/ instead? That's ignored by the updater AFAIK.

Describe alternatives you've considered

Modifying/configuring the updater to exclude the path apps/recognize/models? Don't know if that's a supported feature.

jplitza avatar Aug 12 '23 18:08 jplitza

FTR: I created nextcloud/updater#484 about the symlink support for libtensorflow.so

jplitza avatar Aug 12 '23 18:08 jplitza

Yes, the path for storing the models is not ideal at the moment. It's the same for the translate, stt_whisper and llm apps.

a) The models get removed everytime an update comes in and b) they get backupped unnecessarily.

The problem with storing them in data/ is that there is no hook for the uninstall event in nextcloud, which means recognize will not be able to remove the models when it's uninstalled :/

marcelklehr avatar Aug 15 '23 14:08 marcelklehr

I don't mind the delay that much personally because I use the CLI updater, but it is a bit unfortunate because I don't have that much storage space available. That means that after one or two updates, I need to go into the backups and manually delete them, or I run completely out of disk.

NotAFile avatar Oct 08 '23 18:10 NotAFile