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

Fix problem with camera on Android TV #15163

Merged
merged 3 commits into from
Dec 13, 2017

Conversation

lihanchen
Copy link
Contributor

For a scenario of using a usb external camera, we should use Camera API 2 according to the definition:
Camera1 API is framework API that had been created to support HALv1.x
Camera2 API is a new API that is meant for HALv3.x.

However, CameraCharacteristics.INFO_SUPPORTED_HARDWARE_LEVEL_FULL) should return false as USB camera doesn' support all the necessary capabilities.
https://developer.android.com/reference/android/hardware/camera2/CameraMetadata.html#INFO_SUPPORTED_HARDWARE_LEVEL_FULL

This is a work around to use facing == CameraCharacteristics.LENS_FACING_EXTERNAL to detect usb external camera and use Camera2 API instead.

@tensorflow-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@caisq caisq requested a review from andrewharp December 7, 2017 01:53
@caisq caisq self-assigned this Dec 7, 2017
@caisq caisq added the awaiting review Pull request awaiting review label Dec 7, 2017
@andrewharp andrewharp added the kokoro:force-run Tests on submitted change label Dec 13, 2017
@andrewharp
Copy link
Contributor

@tensorflow-jenkins Test this, please.

@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Dec 13, 2017
@andrewharp
Copy link
Contributor

Merging as the failures have no dependency on the Android demo

@andrewharp andrewharp merged commit 18c864c into tensorflow:master Dec 13, 2017
@lihanchen lihanchen deleted the AndroidTVUSBCamera branch December 13, 2017 21:47
@lihanchen lihanchen restored the AndroidTVUSBCamera branch December 13, 2017 21:48
@lihanchen lihanchen deleted the AndroidTVUSBCamera branch December 13, 2017 21:48
@lihanchen lihanchen restored the AndroidTVUSBCamera branch December 13, 2017 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review Pull request awaiting review cla: yes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants