Re: [Shotwell] Adding face recognition to shotwell

2018-07-21 Thread Narendra Acharya via shotwell-list
Hi, I modified the DBus code to use activation for launching the facedetect process. The build and install process has not been updated to install the DBus service file at the right place though. I presume there needs to be a meson directive to do that.As for the DNN model, should we create a

Re: [Shotwell] Adding face recognition to shotwell

2018-07-18 Thread Narendra Acharya via shotwell-list
Hi Jens, I agree that we should not be forcing the DNN model download for everyone. It could be an optional 'data' package which is required only for face recognition and not detection. This data package could probably be under a different license like you said.So I will assume the basic

Re: [Shotwell] Adding face recognition to shotwell

2018-07-18 Thread Jens Georg
So do we now create 2 packages out of the shotwell tree? How would the facedetect package be published to various distributions? And should the facedetect binary become a service which can be launched by shotwell, or maybe by the OS? You should not care about this at all. Either me (for

Re: [Shotwell] Adding face recognition to shotwell

2018-07-17 Thread Norbert Preining via shotwell-list
Hi all, > I am not familiar with flatpak, so will read up and see how I can deploy this > service using it.So the preferred method seems to be separate package for the > facedetect binary which comes with the DNN model. I think a separate package sounds fine, but I would recommend against

Re: [Shotwell] Adding face recognition to shotwell

2018-07-17 Thread Narendra Acharya via shotwell-list
Hi Andreas, I am not familiar with flatpak, so will read up and see how I can deploy this service using it.So the preferred method seems to be separate package for the facedetect binary which comes with the DNN model. Thanks,Narendra On Tuesday, 17 July, 2018, 12:44:49 PM IST, Andreas

Re: [Shotwell] Adding face recognition to shotwell

2018-07-17 Thread Andreas Brauchli
Hi Narendra, On Mon, 2018-07-16 at 15:28 +, Narendra Acharya via shotwell-list wrote: > Hi, > > As mentioned in my recent posts, I am working on better integration > of face detection (via DBus) and add face recognition as the next > step. The DBus connection cleanup is mostly done in my >

[Shotwell] Adding face recognition to shotwell

2018-07-16 Thread Narendra Acharya via shotwell-list
Hi, As mentioned in my recent posts, I am working on better integration of face detection (via DBus) and add face recognition as the next step. The DBus connection cleanup is mostly done in my fork:  https://gitlab.gnome.org/nma83/shotwell/commits/wip/faces.In order to get to face recognition,