search

BrandonJoffe / home_surveillance

Home surveillance system with facial recognition
1.22k stars 384 forks source link

Mobile alerts #28

Closed mathieuduperre closed 7 years ago

BrandonJoffe commented 7 years ago

Great work @mathieuduperre before I take a closer look at the code, is there any way you could summarise in a paragraph what changes you made?

mathieuduperre commented 7 years ago

hi Brandon, i hit the pull request by mistake, was trying to pull something a colleague had implemented. I'm actually not 100% sure you will want to add our code but when we're ready i'll definetely let you know. if you do a pull at the moment, you would add logging capabilities, a log directory with log files where every steps are logged based on log level. that can be sent to logstash and whats' not, goal being to deploy many many containers with that running, you want to have a single log repo to manage them all. the next drop in a few days will have alarming capabilities to send sms message to cellphone. i'm also fixing the remove camera feature. last piece will be a "learn" button to enable learning people's face from the camera feed. Using flat file has been really hard as the input is only supporting a few format and the whole "teach database" process seems to crash the software mi-way (confirmed by other folsk in the thread). anyway found a way to manually teach and clean db, so will implement buttons to get those going.

ttyl!

BrandonJoffe commented 7 years ago

Ok no problem, sounds great!

There are certainly tones of features to be added and there are a few bugs which need to be fixed (unfortunately time has gotten the better of me).

Keep up the good work :)

Brandon

On 20 Mar 2017, at 23:08, mathieuduperre notifications@github.com wrote:

hi Brandon, i hit the pull request by mistake, was trying to pull something a colleague had implemented. I'm actually not 100% sure you will want to add our code but when we're ready i'll definetely let you know. if you do a pull at the moment, you would add logging capabilities, a log directory with log files where every steps are logged based on log level. that can be sent to logstash and whats' not, goal being to deploy many many containers with that running, you want to have a single log repo to manage them all. the next drop in a few days will have alarming capabilities to send sms message to cellphone. i'm also fixing the remove camera feature. last piece will be a "learn" button to enable learning people's face from the camera feed. Using flat file has been really hard as the input is only supporting a few format and the whole "teach database" process seems to crash the software mi-way (confirmed by other folsk in the thread). anyway found a way to manually teach and clean db, so will implement buttons to get those going.

ttyl!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/BrandonJoffe/home_surveillance/pull/28#issuecomment-287898608, or mute the thread https://github.com/notifications/unsubscribe-auth/ATLLibKLKW_jlRCqIVxPRBPNcR_8IVmYks5rnurlgaJpZM4Migsf.