People count doesn't respect minimum recognized faces

[waclaw66]

The bug

Faces are shown according "Minimum recognized faces" settings, but People count is not affected, it still shows the same count.
If minimum is set to 1, faces and count is shown as 161,
if minimum is set to 2, 138 faces is shown, count is still 161.

The OS that Immich Server is running on

Fedora 40

Version of Immich Server

v1.117.0

Version of Immich Mobile App

not relevant

Platform with the issue

• Server
• Web
• Mobile

Your docker-compose.yml content

not relevant

Your .env content

not relevant

Reproduction steps

Relevant log output

No response

Additional information

No response

[jedi04]

I tried this and I was also getting the same issue. But when you re run Facial Recognition job for all the assets. It solves this issue.

The query used to fetch count does not consider this option[Minimum Recognized Faces]

immich/server/src/repositories/person.repository.ts

Line 248 in 94d213b

.having('COUNT(face.assetId) != 0')

While the query which is used to fetch faces uses this option and that's why count does not match the faces

immich/server/src/repositories/person.repository.ts

Line 105 in 94d213b

.having("person.name != '' OR COUNT(face.assetId) >= :faces", { faces: options?.minimumFaceCount || 1 })

[jedi04]

We can use the same filter while fetching the count also. Can i take up this issue?

[C-Otto]

@jedi04 sure, go ahead!