-
-
Notifications
You must be signed in to change notification settings - Fork 46
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
Able or how to view enable face recognition data/tags to be accessed/viewed globally? #555
Comments
I second this feature request |
@guystreeter I read through both of those issues/feature requests, and although they have some marginal overlap, I would say that I have tried to highlight the main points of the issue where they haven't.
These things would seem to be central to creating a meaningful use-case and implementation of this feature. Something that concerns me more though, is the fact that the issues #413 and #377 seem to be 2 years old, without any indication that anything has or will be even considered beyond "it is something we hope to add in the future". Personally, I think this is a feature that would put this project head and shoulders above other similar projects. I have great faith in this project, but I think maybe a trick is being missed here that could really be important. This and being able to add face tags manually, I don't see how these 2 features aren't high on development backlog. Of course I'm saying this without understanding what the project workload is and dev pressures of course, I'm not privy to any of that. I've been trying to see from looking through the repo how I could contribute, but I don't know how without being an (very good) PHP developer. |
+1 for this issue. Would love to see such features. |
Is there any update to this feature yet? +1 I'd really something similar to this to be implemented. EDIT: I just had a silly idea that I might try just for my own use. Take out the "user" part from the persons and images table. |
I've checked the linked issues and seen nothing, again, years later. It also seems like development on this project has slowed down, is this even being worked on? |
Hello everyone, In 0.9.10 for nc25, add a command to create people's albums. This command is in response to issue #292, but also contributes to part of this report. Please, note that we work according to the nextcloud api and their applications. Although we would like it, we cannot do everything that our users expect (And that I personally would love to see it too) if we do not have the correct tools to do it. I still have my doubts about this (what happens if two users share an album with the same name?). Don't know. Partly for this reason it is a separate command, and it does not work automatically |
Thanks for this response, updates, and reference to the other ticket. 😊 In terms of how to resolve the "two users share an album with the same name", then maybe perhaps there could be some prefix or postfix or user-specific UUID on the Album? This might not need to be a UI-visible ID, but maybe of the Album in the back-end, because personally I don't mind if 2 Albums are named "Family Pictures" on the front-end, if on the backend the albums are named "janet_family_pictures" and "andrew_family_pictures". So perhaps this might solve the clashing of duplicates there, assuming a unique user cannot have duplicates within their own account? Another option might be just to directly specify who shared it in the UI, like "Family Pictures [Janet]" and "Family Pictures [Andrew]" being the album names, with the "[UserName]" of the name being the UUID unique qualifier? I dunno, I'm kinda spitballing here. Thanks again for the response. I do love this amazing project, and use it for my self-hosted family pictures project. |
It is that it does not exist officially yet 😅 , this is for Nextcloud 25 that has not yet been released as stable. 🙈
It is not a technical question. Note that it's just a peculiar use of a nextcloud feature for which it wasn't intended. So, since NC25, this application allows you to create the album with people, however all the features you expect from it, we cannot control it. For example I can not indicate that the album was created by an application, I cannot indicate that it should not be edited, and if you edit the album, the app has no way of knowing, and any changes you make will be reverted on the next run of the command. There are many unresolved things. |
Oh I see, I know nothing about the Nextcloud API but that does sound like a complicated set of contraints to navigate (And I'm sure those examples are a drop in the sea). Either way, I think you've achieved amazing things with this project and I genuinely hope that you can do more with it. I looked at a lot of projects for photo management, and THIS project coupled with Nextcloud makes it simply amazing. I honestly feel that this project is so close to greatness, but I work in IT myself (Experience from many years, and having my own successful IT consultancy company) and I sympathise that even simple things are not simple to implement. I hope I can contribute one day, I've looked into some of the source of this project and its out of my remit for this moment, and the intrinsic knowledge you have of nextcloud APIs is something I'd have to work on as well. |
PREFACE
First of all, I want to preface this by saying I think that this project is AMAZING!
I'm trying to build an online album for my family, where photos can be viewed and aggregated and shared.
I've been spending months testing different solutions, and this project coupled with NextCloud itself hits so many sweet spots, its ALMOST perfect.
I've been spending about a week getting the face recognition function working on a new docker install (On unraid 6.9.2), and it has been a real pain to get working, but I finally got everything working, added images and finally got them indexed and recognised/tagged. I think its great.
That being said...
I think that the workflow of face recognition data/tags being PER USER with no ability to share globally (That I can find), seems to undermine some the purpose of doing all of this. I say this understanding that my use case is not every use case and situation, and that some people may want this workflow and behaviour. Still, in a use case where a "group" or family is tagging pictures together, it seems to me that this would be quite important, especially as its stored in a central database.
So onto the issue template itself...
Expected behaviour
It would be great to have the ability to enable face recognition tags to be globally viewed.
To be able to define who can tag names on photos globally would be "perfect world", but just being able to view would be great.
In a "perfect world", have the ability to set permission groups to:
Personally, at this time I would find it amazing just to be able to view other peoples tagged names, even if they cannot be edited, but a permission based nuanced approach would be chefs kiss.
I understand that this would bring issues around where the file is stored, but I have found the "hidden options" to tag photos in shared storage, and used them hoping that they could be viewed by different users, but alas no.
So I would assume that only photos in shared storage locations would be able to have the tags shared.
Actual behaviour
At the moment, any user that tags a photo can view the name tags, and see them in the "Persons" sidebar, but this data is hidden to other users.
Steps to reproduce
Server configuration
Operating system:
Unraid 6.9.2, running Docker.
Pdlib version:
Unsure at this time, I will find this out and update this issue.
I'm using the Docker image in this issue.
I have version 0.9.1 of Face Recognition installed.
How is DLib installed: Make sure it is working correctly with this tool
In the Docker container, link to installation line in Dockerfile.
How is PDlib installed: Make sure it is working correctly with this tool
In the Docker container, link to installation line in Dockerfile.
PHP version:
PHP 8.0.14 (cli) (built: Dec 21 2021 20:43:00) ( NTS )
Web server:
apache2 stable, 2.4.51-1~deb11u1
Database:
Using SQL Lite for testing, will definitely move to MySQL later for bigger use.
Nextcloud version:
23.0.0, Docker installed.
The text was updated successfully, but these errors were encountered: