[BUG] Merging one unnamed face with one named face should choose the named faces name #1984

Closed
opened 2026-02-05 04:40:37 +03:00 by OVERLORD · 3 comments
Owner

Originally created by @jhbruhn on GitHub (Jan 15, 2024).

The bug

When merging faces directly from an unnamed faces context menu in the face selection overview, and then choosing to merge with an already existing, named faced, the resulting face is an unnamed face. This is not what I expected, as I would expect the named faces name to be chosen.

The OS that Immich Server is running on

Docker on Synology NAS

Version of Immich Server

1.92.1

Version of Immich Mobile App

?

Platform with the issue

  • Server
  • Web
  • Mobile

Your docker-compose.yml content

default docker compose

Your .env content

default env

Reproduction steps

1. Have a named face
2. Have an unnamed face
3. Go to face overview
4. Click on unnamed faces context menu and choose "Merge face"
5. Chose your previously created unnamed face
6. The resulting face is an unnamed face

Additional information

No response

Originally created by @jhbruhn on GitHub (Jan 15, 2024). ### The bug When merging faces directly from an unnamed faces context menu in the face selection overview, and then choosing to merge with an already existing, named faced, the resulting face is an unnamed face. This is not what I expected, as I would expect the named faces name to be chosen. ### The OS that Immich Server is running on Docker on Synology NAS ### Version of Immich Server 1.92.1 ### Version of Immich Mobile App ? ### Platform with the issue - [ ] Server - [X] Web - [ ] Mobile ### Your docker-compose.yml content ```YAML default docker compose ``` ### Your .env content ```Shell default env ``` ### Reproduction steps ```bash 1. Have a named face 2. Have an unnamed face 3. Go to face overview 4. Click on unnamed faces context menu and choose "Merge face" 5. Chose your previously created unnamed face 6. The resulting face is an unnamed face ``` ### Additional information _No response_
Author
Owner

@AngelPone commented on GitHub (Jan 15, 2024):

Actually, there is a switch you can use to merge the unnamed face to a named face

CleanShot 2024-01-15 at 12 51 28

@AngelPone commented on GitHub (Jan 15, 2024): Actually, there is a switch you can use to merge the unnamed face to a named face ![CleanShot 2024-01-15 at 12 51 28](https://github.com/immich-app/immich/assets/32930283/845edb82-36e6-465c-a895-788a7698270d)
Author
Owner

@jhbruhn commented on GitHub (Jan 15, 2024):

That's nice to know, however I think it would make sense for immich to pre-select the named face as the resulting face.

@jhbruhn commented on GitHub (Jan 15, 2024): That's nice to know, however I think it would make sense for immich to pre-select the named face as the resulting face.
Author
Owner

@jrasm91 commented on GitHub (Sep 9, 2024):

The name is now kept on merge.

@jrasm91 commented on GitHub (Sep 9, 2024): The name is now kept on merge.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#1984