Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error merging faces with app. #689

Open
johnhelt opened this issue Aug 6, 2023 · 5 comments
Open

Error merging faces with app. #689

johnhelt opened this issue Aug 6, 2023 · 5 comments

Comments

@johnhelt
Copy link

johnhelt commented Aug 6, 2023

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

  1. Go to people (I use memories). Select a person.
  2. Select the options menu (the 3 dots ...). Chose merge with a different person
  3. 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

<?xml version="1.0" encoding="utf-8"?>

2

<d:error xmlns:d="DAV:" xmlns:s="http://sabredav.org/ns">

3

  <s:exception>Sabre\DAV\Exception\NotFound</s:exception>

4

  <s:message>File not found: recognize in 'root'</s:message>

5

</d:error>

6

​```
</details>
@matiasdelellis
Copy link
Owner

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

@kabaga
Copy link

kabaga commented Sep 4, 2023

I got the same issue

@matiasdelellis
Copy link
Owner

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

@kabaga
Copy link

kabaga commented Sep 4, 2023

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
Copy link
Owner

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..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants