matiasdelellis / facerecognition

Nextcloud app that implement a basic facial recognition system.
GNU Affero General Public License v3.0
510 stars 46 forks source link

Rename big cluster results in outage of nextcloud #753

Open sirdrakeistda opened 4 months ago

sirdrakeistda commented 4 months ago

Expected behaviour

Renaming big cluster should work without problems.

Actual behaviour

Ttrying to name a big cluster results in an outage of the nextcloud.

Steps to reproduce

1.Oupload many images of the same person (This could happen on a big event like birthday, party, wedding, ... were many images are created) 2.wait until facrecognition creates clusters of the person with >1000 images 3.Open facerecognition in Nextcloud and try to name the clusters -> loading the naming window takes very long and could result in a complete outage of the nextcloud until a restart

I'm not sure what facerecognition does in the background, so don't take my solution ideas too serious. For me it seems like all images are loaded in the background, even tough they are not displayed. Two things would be nice to solve this problem:

Server configuration

sirdrakeistda commented 4 months ago

For all people who face this issue: A workaround is the name person cluster option in memories (not merge) instead of the naming option in facrecognition itself. This option has a much better performance and wont crash the server.