-
Notifications
You must be signed in to change notification settings - Fork 629
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
facerecognition: change to FACE_MODEL=3 and adjust memory values #4742
Conversation
Signed-off-by: Simon L <szaimen@e.mail.de>
@matiasdelellis can we also lower the limit in
Or does it need to stay as high due to
|
The absolute minimum is 512 MB that you have to set to allow it to work. but if the user has a lot of images (50 thousand?) they will probably need more memory to perform the face clustering step, therefore we must leave a reasonable value to start. I will try to evaluate but maybe 2GB could be something very reasonable value.
1G, it is a mere formality but we must assign it.
It's a lot, but it will work without problems. |
p.s: You can also increase the memory only for the commands and leave the default configuration.
|
Signed-off-by: Simon L <szaimen@e.mail.de>
all right, adjusted :) |
This is now released with v9.0.0 Beta. Testing and feedback is welcome! See https://github.com/nextcloud/all-in-one#how-to-switch-the-channel |
For #4721 (reply in thread)