Skip to content
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

obscuracam 2017! #114

Merged
merged 15 commits into from
Aug 4, 2017
Merged

obscuracam 2017! #114

merged 15 commits into from
Aug 4, 2017

Conversation

n8fr8
Copy link
Member

@n8fr8 n8fr8 commented Aug 1, 2017

A few small error handling fixes, removal of the Google Play Vision FaceDetect (which doesn't work that well it turns out), and an update of the ffmpeg binary

@DavidMOliver
Copy link
Collaborator

Wait - you're removing ALL face detection?

@n8fr8
Copy link
Member Author

n8fr8 commented Aug 1, 2017

We are using the more reliable local Android open-source FaceDetection, which is built into the OS, vs using it in combination with the closed source Google Play cloud-enhanced face detection, which hasn't proven to be better, and often worse.

@n8fr8
Copy link
Member Author

n8fr8 commented Aug 1, 2017

(In the long run, I want to integrate OpenCV face detect libraries opencv.org

@DavidMOliver
Copy link
Collaborator

ah - apologies - groovy

- something isn't working with the libJpegRedaction library, we'll just use inmemory bitmaps for now
- also improved the flow a bit for image viewing and processing
- removed our own ffmpeg impl, in favor of: http://writingminds.github.io/ffmpeg-android-java/
- cleaned up menus and "open" at end
- moved to drawbox redacting for now, instead of custom redact filter
@N-Pex N-Pex merged commit 6d9d001 into version_2017 Aug 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants