immich-app/immich
1
0
Fork 0
mirror of https://github.com/immich-app/immich.git synced 2026-07-25 14:00:45 +03:00
Code Issues 783 Packages Projects Releases 22 Wiki Activity

Facial recognition bug #4145

New Issue
Closed
opened 2026-02-05 09:47:27 +03:00 by OVERLORD · 8 comments
OVERLORD commented 2026-02-05 09:47:27 +03:00
Owner
Copy Link

Originally created by @pyccl on GitHub (Aug 24, 2024).

The bug

1: The actual number of facial recognition is less than what is displayed, with 97 in reality and 113 missing. After partially hiding some faces, there are actually 36 and 52 displayed.

2: The last character image on the "Show and Hide Characters" page is smaller than the other characters.

The OS that Immich Server is running on

Windows docker desktop 4.30.0

Version of Immich Server

v1.112.1

Version of Immich Mobile App

v

Platform with the issue

  • Server
  • Web
  • Mobile

Your docker-compose.yml content

#
# WARNING: Make sure to use the docker-compose.yml of the current release:
#
# https://github.com/immich-app/immich/releases/latest/download/docker-compose.yml
#
# The compose file on main may not be compatible with the latest release.
#

name: immich

services:
  immich-server:
    container_name: immich_server
    image: ghcr.io/immich-app/immich-server:${IMMICH_VERSION:-release}
    # extends:
    #   file: hwaccel.transcoding.yml
    #   service: cpu # set to one of [nvenc, quicksync, rkmpp, vaapi, vaapi-wsl] for accelerated transcoding
    volumes:
      # Do not edit the next line. If you want to change the media storage location on your system, edit the value of UPLOAD_LOCATION in the .env file
      - ${UPLOAD_LOCATION}:/usr/src/app/upload
      - /etc/localtime:/etc/localtime:ro
      - "E:/My photo:/mnt/media/myphoto"
    env_file:
      - .env
    ports:
      - 2283:3001
    depends_on:
      - redis
      - database
    restart: always
    healthcheck:
      disable: false

  immich-machine-learning:
    container_name: immich_machine_learning
    # For hardware acceleration, add one of -[armnn, cuda, openvino] to the image tag.
    # Example tag: ${IMMICH_VERSION:-release}-cuda
    image: ghcr.io/immich-app/immich-machine-learning:${IMMICH_VERSION:-release}
    # extends: # uncomment this section for hardware acceleration - see https://immich.app/docs/features/ml-hardware-acceleration
    #   file: hwaccel.ml.yml
    #   service: cpu # set to one of [armnn, cuda, openvino, openvino-wsl] for accelerated inference - use the `-wsl` version for WSL2 where applicable
    volumes:
     # - model-cache:/cache
      - "E:/immich/cache:/cache"
    env_file:
      - .env
    restart: always
    healthcheck:
      disable: false

  redis:
    container_name: immich_redis
    image: m.daocloud.io/docker.io/redis:6.2-alpine@sha256:e3b17ba9479deec4b7d1eeec1548a253acc5374d68d3b27937fcfe4df8d18c7e
    healthcheck:
      test: redis-cli ping || exit 1
    restart: always

  database:
    container_name: immich_postgres
    image: m.daocloud.io/docker.io/tensorchord/pgvecto-rs:pg14-v0.2.0@sha256:90724186f0a3517cf6914295b5ab410db9ce23190a2d9d0b9dd6463e3fa298f0
    environment:
      POSTGRES_PASSWORD: ${DB_PASSWORD}
      POSTGRES_USER: ${DB_USERNAME}
      POSTGRES_DB: ${DB_DATABASE_NAME}
      POSTGRES_INITDB_ARGS: '--data-checksums'
    volumes:
      # Do not edit the next line. If you want to change the database storage location on your system, edit the value of DB_DATA_LOCATION in the .env file
      #- ${DB_DATA_LOCATION}:/var/lib/postgresql/data
      - pgdata:/var/lib/postgresql/data
    healthcheck:
      test: pg_isready --dbname='${DB_DATABASE_NAME}' --username='${DB_USERNAME}' || exit 1; Chksum="$$(psql --dbname='${DB_DATABASE_NAME}' --username='${DB_USERNAME}' --tuples-only --no-align --command='SELECT COALESCE(SUM(checksum_failures), 0) FROM pg_stat_database')"; echo "checksum failure count is $$Chksum"; [ "$$Chksum" = '0' ] || exit 1
      interval: 5m
      start_interval: 30s
      start_period: 5m
    command: ["postgres", "-c", "shared_preload_libraries=vectors.so", "-c", 'search_path="$$user", public, vectors', "-c", "logging_collector=on", "-c", "max_wal_size=2GB", "-c", "shared_buffers=512MB", "-c", "wal_compression=on"]
    restart: always

volumes:
  model-cache:
  pgdata:

Your .env content

# You can find documentation for all the supported env variables at https://immich.app/docs/install/environment-variables

# The location where your uploaded files are stored
UPLOAD_LOCATION=./library
# The location where your database files are stored
DB_DATA_LOCATION=./postgres

# To set a timezone, uncomment the next line and change Etc/UTC to a TZ identifier from this list: https://en.wikipedia.org/wiki/List_of_tz_database_time_zones#List
# TZ=Asia/Shanghai

# The Immich version to use. You can pin this to a specific version like "v1.71.0"
IMMICH_VERSION=release

# Connection secret for postgres. You should change it to a random password
# Please use only the characters `A-Za-z0-9`, without special characters or spaces
DB_PASSWORD=postgres

# The values below this line do not need to be changed
###################################################################################
DB_USERNAME=postgres
DB_DATABASE_NAME=immich

Reproduction steps

1.
2.
3.
...

Relevant log output

No response

Additional information

No response

Originally created by @pyccl on GitHub (Aug 24, 2024). ### The bug 1: The actual number of facial recognition is less than what is displayed, with 97 in reality and 113 missing. After partially hiding some faces, there are actually 36 and 52 displayed. 2: The last character image on the "Show and Hide Characters" page is smaller than the other characters. ### The OS that Immich Server is running on Windows docker desktop 4.30.0 ### Version of Immich Server v1.112.1 ### Version of Immich Mobile App v ### Platform with the issue - [ ] Server - [X] Web - [ ] Mobile ### Your docker-compose.yml content ```YAML # # WARNING: Make sure to use the docker-compose.yml of the current release: # # https://github.com/immich-app/immich/releases/latest/download/docker-compose.yml # # The compose file on main may not be compatible with the latest release. # name: immich services: immich-server: container_name: immich_server image: ghcr.io/immich-app/immich-server:${IMMICH_VERSION:-release} # extends: # file: hwaccel.transcoding.yml # service: cpu # set to one of [nvenc, quicksync, rkmpp, vaapi, vaapi-wsl] for accelerated transcoding volumes: # Do not edit the next line. If you want to change the media storage location on your system, edit the value of UPLOAD_LOCATION in the .env file - ${UPLOAD_LOCATION}:/usr/src/app/upload - /etc/localtime:/etc/localtime:ro - "E:/My photo:/mnt/media/myphoto" env_file: - .env ports: - 2283:3001 depends_on: - redis - database restart: always healthcheck: disable: false immich-machine-learning: container_name: immich_machine_learning # For hardware acceleration, add one of -[armnn, cuda, openvino] to the image tag. # Example tag: ${IMMICH_VERSION:-release}-cuda image: ghcr.io/immich-app/immich-machine-learning:${IMMICH_VERSION:-release} # extends: # uncomment this section for hardware acceleration - see https://immich.app/docs/features/ml-hardware-acceleration # file: hwaccel.ml.yml # service: cpu # set to one of [armnn, cuda, openvino, openvino-wsl] for accelerated inference - use the `-wsl` version for WSL2 where applicable volumes: # - model-cache:/cache - "E:/immich/cache:/cache" env_file: - .env restart: always healthcheck: disable: false redis: container_name: immich_redis image: m.daocloud.io/docker.io/redis:6.2-alpine@sha256:e3b17ba9479deec4b7d1eeec1548a253acc5374d68d3b27937fcfe4df8d18c7e healthcheck: test: redis-cli ping || exit 1 restart: always database: container_name: immich_postgres image: m.daocloud.io/docker.io/tensorchord/pgvecto-rs:pg14-v0.2.0@sha256:90724186f0a3517cf6914295b5ab410db9ce23190a2d9d0b9dd6463e3fa298f0 environment: POSTGRES_PASSWORD: ${DB_PASSWORD} POSTGRES_USER: ${DB_USERNAME} POSTGRES_DB: ${DB_DATABASE_NAME} POSTGRES_INITDB_ARGS: '--data-checksums' volumes: # Do not edit the next line. If you want to change the database storage location on your system, edit the value of DB_DATA_LOCATION in the .env file #- ${DB_DATA_LOCATION}:/var/lib/postgresql/data - pgdata:/var/lib/postgresql/data healthcheck: test: pg_isready --dbname='${DB_DATABASE_NAME}' --username='${DB_USERNAME}' || exit 1; Chksum="$$(psql --dbname='${DB_DATABASE_NAME}' --username='${DB_USERNAME}' --tuples-only --no-align --command='SELECT COALESCE(SUM(checksum_failures), 0) FROM pg_stat_database')"; echo "checksum failure count is $$Chksum"; [ "$$Chksum" = '0' ] || exit 1 interval: 5m start_interval: 30s start_period: 5m command: ["postgres", "-c", "shared_preload_libraries=vectors.so", "-c", 'search_path="$$user", public, vectors', "-c", "logging_collector=on", "-c", "max_wal_size=2GB", "-c", "shared_buffers=512MB", "-c", "wal_compression=on"] restart: always volumes: model-cache: pgdata: ``` ### Your .env content ```Shell # You can find documentation for all the supported env variables at https://immich.app/docs/install/environment-variables # The location where your uploaded files are stored UPLOAD_LOCATION=./library # The location where your database files are stored DB_DATA_LOCATION=./postgres # To set a timezone, uncomment the next line and change Etc/UTC to a TZ identifier from this list: https://en.wikipedia.org/wiki/List_of_tz_database_time_zones#List # TZ=Asia/Shanghai # The Immich version to use. You can pin this to a specific version like "v1.71.0" IMMICH_VERSION=release # Connection secret for postgres. You should change it to a random password # Please use only the characters `A-Za-z0-9`, without special characters or spaces DB_PASSWORD=postgres # The values below this line do not need to be changed ################################################################################### DB_USERNAME=postgres DB_DATABASE_NAME=immich ``` ### Reproduction steps ```bash 1. 2. 3. ... ``` ### Relevant log output _No response_ ### Additional information _No response_
OVERLORD commented 2026-02-05 09:47:30 +03:00
Author
Owner
Copy Link

@pyccl commented on GitHub (Aug 24, 2024):

There are actually 97, but 113 are displayed. After hiding, there are actually 36, but 52 are displayed.

@pyccl commented on GitHub (Aug 24, 2024): There are actually 97, but 113 are displayed. After hiding, there are actually 36, but 52 are displayed.
OVERLORD commented 2026-02-05 09:47:32 +03:00
Author
Owner
Copy Link

@phantomwork commented on GitHub (Aug 24, 2024):

Can you provide more details including images

@phantomwork commented on GitHub (Aug 24, 2024): Can you provide more details including images
OVERLORD commented 2026-02-05 09:47:33 +03:00
Author
Owner
Copy Link

@pyccl commented on GitHub (Aug 24, 2024):

1

@pyccl commented on GitHub (Aug 24, 2024): ![1](https://github.com/user-attachments/assets/2fe545f9-a327-4f30-a4cf-29fd9209c331)
OVERLORD commented 2026-02-05 09:47:35 +03:00
Author
Owner
Copy Link

@Cookie987 commented on GitHub (Nov 21, 2024):

Also have this problem:
image

@Cookie987 commented on GitHub (Nov 21, 2024): Also have this problem: ![image](https://github.com/user-attachments/assets/3bbe91dd-c744-4999-820f-3062afba4453)
OVERLORD commented 2026-02-05 09:47:37 +03:00
Author
Owner
Copy Link

@pyccl commented on GitHub (Nov 24, 2024):

Also have this problem:
image

这个问题一直就没解决😂😂😂

@pyccl commented on GitHub (Nov 24, 2024): > Also have this problem: > ![image](https://github.com/user-attachments/assets/3bbe91dd-c744-4999-820f-3062afba4453) > > 这个问题一直就没解决😂😂😂
OVERLORD commented 2026-02-05 09:47:39 +03:00
Author
Owner
Copy Link

@jrasm91 commented on GitHub (Jun 24, 2025):

This looks to be caused by different filters between these two queries:

https://github.com/immich-app/immich/blob/main/server/src/services/person.service.ts#L64-L77

@mertalev is this something you could take a look at? I think the getNumberOfPeople query should also take minFaceCount into account so the queries target the same set of people and the statistics match up.

@jrasm91 commented on GitHub (Jun 24, 2025): This looks to be caused by different filters between these two queries: https://github.com/immich-app/immich/blob/main/server/src/services/person.service.ts#L64-L77 @mertalev is this something you could take a look at? I think the `getNumberOfPeople` query should also take `minFaceCount` into account so the queries target the same set of people and the statistics match up.
OVERLORD commented 2026-02-05 09:47:40 +03:00
Author
Owner
Copy Link

@mertalev commented on GitHub (Jun 24, 2025):

Yes, that seems to be the issue. It might be best to fuse that query with the getAllForUser query. getAllForUser is in the unfortunate position of being a paginated query with aggregations, so the cost of including the total count in that query is much lower than aggregating again in a separate query, along with making a mismatch impossible.

@mertalev commented on GitHub (Jun 24, 2025): Yes, that seems to be the issue. It might be best to fuse that query with the `getAllForUser` query. `getAllForUser` is in the unfortunate position of being a paginated query with aggregations, so the cost of including the total count in that query is much lower than aggregating again in a separate query, along with making a mismatch impossible.
OVERLORD commented 2026-02-05 09:47:41 +03:00
Author
Owner
Copy Link

@jrasm91 commented on GitHub (Jun 24, 2025):

Yeah, I was wondering about doing something like that. I actually started to work on it, but decided better of it 😅

@jrasm91 commented on GitHub (Jun 24, 2025): Yeah, I was wondering about doing something like that. I actually started to work on it, but decided better of it 😅
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
accessibility
changelog:enhancement
changelog:security
changelog:skip
changelog:translation
cli
date-time
dependencies
documentation
external-library
format
good first issue
mobile-beta
mobile-beta
mobile-beta
needs-answer
nice to have
pull-request
Mirrored from GitHub Pull Request
 sharing
tech-debt
📱mobile
🖥️web
🗄️server
🧠machine-learning
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#4145
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?