search

powderblock / DealWithItReddit

Reddit bot. Computer vision for facial detection and transformation. Python, PRAW, & OpenCV.
http://www.reddit.com/user/DealWithItbot/
MIT License
33 stars 3 forks source link

Fix repeated processing of images #8

Closed porglezomp closed 9 years ago

porglezomp commented 9 years ago

This PR fixes both #6 and #7, they're caused by the same piece of code
The old code would process all of them every time it found one collisions, which caused both repeated calculations, and also would use all of the eyes in every one, even if the eyes failed once. This probably also caused some of the rate limit errors.

porglezomp commented 9 years ago

It should do that, yes

porglezomp commented 9 years ago

Make sure you test this code

powderblock commented 9 years ago

Update README.md

porglezomp commented 9 years ago

You can handle that while you're merging, I'm not sure that's my job? Maybe it is.