Mert
Mert
Just checking in! I'm realizing that the scope of this PR is really big, which makes it difficult to work on and keep up-to-date with main. There are some related...
Totally! One thing I'll mention: I don't think it's a good idea to do the ML call in the facial recognition job for performance reasons. That job is fixed at...
I've hit an issue with the recognition part. The recognition model actually doesn't use the bounding boxes - it uses facial landmarks (eyes, nose, mouth) to warp the image so...
The solution will involve adding a separate landmark detection model. The face detection model actually does this as well, but we can't use it here: it has a separate limitation...
Following #10371, faces are no longer required to have embeddings. While it'd be ideal to have them for facial recognition, the feature is still useful outside of that and ties...
This PR hasn't been merged and isn't in any release. We'll certainly showcase it in the release notes when it's ready :)
[This](https://github.com/strukturag/libheif/issues/784) seems relevant.
I believe this is an open issue in libheif and stems from the HEIF here being a bit quirky (but not invalid). But per that issue, the actual error seems...
The default for min recognized faces has been increased from 1 to 3, so if the people you notice no longer appear only had 1 or 2 images, then it...
That part is normal. It's a known edge-case that DBSCAN can produce clusters with fewer points than the minimum, but this is benign. It just means there was another person...