Error merging faces with app.

johnhelt commented 1 year ago

Expected behaviour

I want to merge two faces, that are the same person, but in slightly different circumstances (one with shades, one without shades). This should be possible by selecting "people" and "merge with another person" from the options menu (...).

Actual behaviour

I get an error "Failed to move XXXX"

Steps to reproduce

Go to people (I use memories). Select a person.
Select the options menu (the 3 dots ...). Chose merge with a different person
Chose a person to merge with. Observe error.

Server configuration

Operating system: Docker nextcloud running php-fpm-
Pdlib version: 1.0.2, I think.
How is DLib installed: Make sure it is working correctly with this tool I'm running everything through Docker, and I followed your guide..
How is PDlib installed: Make sure it is working correctly with this tool As mentioned above
PHP version: 8.1.19
Web server: Nginx
Database: Mariadb
Nextcloud version: 25.0.6

Client configuration
Browser: Firefox
Operating system: Ubuntu, Android and Windows

Logs

Background task log with debug.

sudo -u apache php occ -vvv face:background_job

``` Insert your background log here ``` I've skipped this, because it didn't show anything interesting, and it seems this bug is not related to the background job. #### Web server error log

Web server error log

Only thing I could see is this at the time I tried to merge: ``` nextcloud-web-1 | 172.22.0.2 - - [06/Aug/2023:18:34:58 +0000] "PROPFIND /remote.php/dav/recognize/REDACTED/faces/5418/ HTTP/1.1" 404 196 "-" "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/116.0" "REDACTED" ```

#### Nextcloud log (data/nextcloud.log)

Nextcloud log

Tail of log after trying the merge: ``` {"reqId":"OFFUGqCUJajn1Hfd9g5M","level":1,"time":"August 06, 2023 17:35:05","remoteAddr":"","user":"--","app":"memories","method":"","url":"--","message":"Memories: Indexing completed successfully","userAgent":"--","version":"25.0.6.1","data":{"app":"memories"}} {"reqId":"bx1owxd8hobhAimOEt5i","level":1,"time":"August 06, 2023 17:55:02","remoteAddr":"","user":"--","app":"memories","method":"","url":"--","message":"Memories: Indexing completed successfully","userAgent":"--","version":"25.0.6.1","data":{"app":"memories"}} {"reqId":"GPBQkpJZ3xmkfMdN1WgB","level":1,"time":"August 06, 2023 18:10:06","remoteAddr":"","user":"--","app":"memories","method":"","url":"--","message":"Memories: Indexing completed successfully","userAgent":"--","version":"25.0.6.1","data":{"app":"memories"}} {"reqId":"L3EyepyxhgkoqTaZkwuJ","level":1,"time":"August 06, 2023 18:30:03","remoteAddr":"","user":"--","app":"mail","method":"","url":"--","message":"No iMIP messages to process.","userAgent":"--","version":"25.0.6.1","data":{"app":"mail"}} {"reqId":"L3EyepyxhgkoqTaZkwuJ","level":1,"time":"August 06, 2023 18:30:06","remoteAddr":"","user":"--","app":"memories","method":"","url":"--","message":"Memories: Indexing completed successfully","userAgent":"--","version":"25.0.6.1","data":{"app":"memories"}} ```

#### Browser log

Browser log

Console log: ``` PROPFIND https://cloud.baebsiville.dk/remote.php/dav/recognize/REDACTED /faces/5418/ [HTTP/2 404 Not Found 51ms] Error: Invalid response: 404 Not Found XHRPROPFIND https://cloud.baebsiville.dk/remote.php/dav/recognize/johnhelt/faces/5418/ [HTTP/2 404 Not Found 51ms] 1 2 3 Sabre\DAV\Exception\NotFound 4 File not found: recognize in 'root' 5 6 ```

matiasdelellis commented 11 months ago

Hi @johnhelt It is pending. That is why the integration is considered experimental. I hope to do it soon..

kabaga commented 11 months ago

I got the same issue

matiasdelellis commented 11 months ago

HI both, In the meantime you can assign the names directly to them. I use that because it's much faster.

kabaga commented 11 months ago

Are you saying that I could use the same name? I tried that in the past, and the person got removed from the People section in Memories.

matiasdelellis commented 11 months ago

Are you saying that I could use the same name?

Yes.

I tried that in the past, and the person got removed from the People section in Memories.

This is a bug in memories but this due recognize backend limitations..

jamesdeeen commented 1 month ago

Hello. Any progress on this issue? For my kids i have many duplicate faces and want to merge them together... I'm running nextcloud 29 AIO

matiasdelellis / facerecognition