Skip to content

Merging recognized faces of different people is cumbersome and error prone #1505

Open
@csware

Description

@csware

Describe the bug
Install the face recognition (great idea to add such a detection!) When trying to merge detected persons, the list is constanly updating and it is highly likely that the order changes and ylou click on the 'wrong' person.

Also, when you open a person, highlight the spotted face in the photos so that it is easier to see whether the correct face was recognized (this would be particularly helpful is several people are on the photo).

Also, quite often a JavaScript error occurs:

TypeError: n.faceDetections.find(...) is undefined
    getFaceCover FaceCoverMixin.js:53
    getFaceCover FaceCoverMixin.js:52
    getCoverStyle FaceCoverMixin.js:76
    H FaceMergeForm.vue:1
    tt VueJS
    H FaceMergeForm.vue:1
    VueJS 62
    click FaceContent.vue:1
    VueJS 4
    onClick ncvuecomponents.js:2
    VueJS 3

To Reproduce
Steps to reproduce the behavior:

  1. Go to /apps/photos/faces/
  2. Select a person
  3. Open the menu and select 'merge'

Expected behavior
The list is static, ordered by the number of associated fotos, and the images are loaded on an best-.effort basis.

Desktop (please complete the following information):

  • OS: Windows
  • Browser Firefox
  • Version 107

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    0. Needs triagePending approval or rejection. This issue is pending approval.bugSomething isn't workingfeature: faces

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions