matiasdelellis / facerecognition

Nextcloud app that implement a basic facial recognition system.
GNU Affero General Public License v3.0
510 stars 46 forks source link

added options to command face:sync-albums #709

Closed wronny closed 5 months ago

wronny commented 10 months ago

Hello, I like facerecognition and its sync-album command. Because I don't want/need to create albums for all persons I added an option to create only albums for selected persons. I also like to create albums with photos containing several persons therefore I added a mode option "Album-per-person" vs "Album-combined". Also added a list persons option to allow to list persons from command line.

Options added to command face:sync-albums: -l, --list_persons : List all person defined for a given user_id -p, --person_name=PERSON_NAME : Sync albums for a given user and person name(s) (separeted using comma). If not used, sync all persons defined by the user -m, --mode=MODE : Album creation mode. Use "Album-per-person" to create one album for each given person via person_name parameter. Use "Album-combined" to create one album for all person names given via person_name parameter. [default: "Album-per-user"]

I hope this is meaningful/helpful also for others. Of course open for feedback / discussion.

Cheers, Marcel

matiasdelellis commented 8 months ago

Hi @wronny Thank you very much for the contribution. I find it very interesting. 😬

I haven't been able to try it yet, because I'm dealing with other issues, but I'll try to do it these days.. πŸ˜„

matiasdelellis commented 5 months ago

Well, I was never able to test it properly, but it's a nice addition, and doesn't change the previous flow.

Thanks again,

matiasdelellis commented 5 months ago

Wow!. I'm just trying it and I love it!! πŸ˜„

Misc fixes on last commit... mainly caps and spaces, and now it works without any arguments (which seems to have never worked! 🀦🏻).