People: Improve naming, renaming, and merging

[alexislefebvre]

I had several images with faces, say person A and B. I wanted to change the tagged person A on one photo, so I changed the name to B. Now all the images with person A belong to person B, it looks like it merged the 2 persons. I think we should add a warning when changing one photo will affect several faces.

Workaround: press the eject button before changing a name:

Side note: congrats that this new feature, this is great!

[jackbrown1993]

Agree, that some sort of confirmation or warning is required. Done this by mistake a couple of times now.

[graciousgrey]

Changing the name without pressing the reject button will rename the person see: https://docs.photoprism.org/user-guide/organize/people/#rename-person. But obviously it is not that clear

[lastzero]

That's exactly why we had the preview for testing. It works as designed. If you rename a person, it's renamed everywhere. Otherwise you need to press the button, so that PhotoPrism knows it's a DIFFERENT person. We'll improve our UI and docs to make this even more clear.

[alexislefebvre]

@graciousgrey Thanks, my bad, I didn't read the doc before trying this new feature.

Here is the context:

The other tabs allow to edit data for only that photo, but changing the person here will affect all the related photos, this is surprising.

[lastzero]

Probably best to disable editing names in the photo edit dialog at all to avoid mistakes.

[graciousgrey]

@alexislefebvre no worries, it's good to have that feedback! Most people don't read the docs at all, so it should be as intuitive as possible :)

[lastzero]

Started a new Development Preview build including the following changes:

• You now have to confirm merging people on the People overview page when you name a person like an existing person.
• Editing existing names in the photo edit dialog is disabled to avoid mistakes.
• Orphan people and faces are removed automatically in the background, so that the count in the navigation always matches the actual search results.

[alexislefebvre]

Thank you.

I tried on the demo. I have set the name for a cluster, then I opened that show media with this person's face. When I pressed the eject button on one photo, it disconnected several photos from the same person.

And we can't mistakenly merge 2 people anymore. 👏