search

pageauc / picamera-motion

Raspberry Pi python PiCamera Lightweight Motion Detection. Includes easy curl script install/upgrade, whiptail admin menu system, single file web server and Rclone for uploading to a variety of web storage services.
91 stars 15 forks source link

Possibly incorrect assignment #2

Closed derefer closed 6 years ago

derefer commented 6 years ago

Hi, I know it's an older library, but shouldn't it be: data1 = data2?

https://github.com/pageauc/picamera-motion/blob/92235548c53c671de49cdc035884d6b428d95688/picamera-motion.py#L172

pageauc commented 6 years ago

Thanks. I have fixed this problem and you can download from github

wget -O picamera-motion.py https://raw.github.com/pageauc/picamera-motion/master/picamera-motion.py

Looks like a brain freeze and lack of good testing. I moved on to other projects and left this behind. Appreciate your feedback. Claude ...

pageauc commented 6 years ago

release 1.3 I did some quick changes to make code python3 compatible and clean up coding. Claude ...

pageauc commented 6 years ago

release 1.5 OK. Did some more code cleanup and simplified. Leaves area for user to adapt if they wish. See pi-timolo repo if user needs more functional implementation.

derefer commented 6 years ago

Thanks for the quick fix!

pageauc commented 6 years ago

Your welcome. Make sure you have latest version 1.5

On Tue, Dec 12, 2017 at 11:08 AM, Ferenc Kovács notifications@github.com wrote:

Thanks for the quick fix!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/pageauc/picamera-motion/issues/2#issuecomment-351118000, or mute the thread https://github.com/notifications/unsubscribe-auth/AFr1ZPOqGY5jYBLgmatjMdD3FwtvqcX2ks5s_rL0gaJpZM4Q-sEf .

-- See my YouTube Channel at http://www.youtube.com/user/pageaucp

pageauc commented 6 years ago

I have improved this by adding rclone sync capability for uploading images to a remote storage service plus easy curl install See links in Readme for more info. Also the pi-timolo project has a very extensive wiki.

https://github.com/pageauc/picamera-motion

On Tue, Dec 12, 2017 at 2:18 PM, Claude Pageau pageauc@gmail.com wrote:

Your welcome. Make sure you have latest version 1.5

On Tue, Dec 12, 2017 at 11:08 AM, Ferenc Kovács notifications@github.com wrote:

Thanks for the quick fix!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/pageauc/picamera-motion/issues/2#issuecomment-351118000, or mute the thread https://github.com/notifications/unsubscribe-auth/AFr1ZPOqGY5jYBLgmatjMdD3FwtvqcX2ks5s_rL0gaJpZM4Q-sEf .

-- See my YouTube Channel at http://www.youtube.com/user/pageaucp

-- See my YouTube Channel at http://www.youtube.com/user/pageaucp