bhargavg1 Reporter
	Description • 10 years ago

See that for front camera we dont show face detection circles even though the HAL/Gecko process these events

From logs

E/QCamera2HWI(  228): [KPI Perf] static void qcamera::QCamera2HardwareInterface::metadata_stream_cb_routine(mm_camera_super_buf_t*, qcamera::QCameraStream*, void*): PROFILE_NUMBER_OF_FACES_DETECTED 1
E/GonkCamera( 1068): Camera detected 1 face(s)
E/GonkCamera( 1068): Camera face[0] appended: id=1, score=72, bound=(-372, -167)-(71, 424)
E/GonkCamera( 1068):     Left eye detected at (-200, 200)
E/GonkCamera( 1068):     Right eye detected at (-185, -25)
E/GonkCamera( 1068):     Mouth detected at (-19, 112)
E/GonkCamera( 1068): DOM OnFacesDetected 1 face(s)

	bhargavg1 Reporter
	Comment 1 • 10 years ago

[Blocking Requested - why for this release]:

	Jason Smith [:jsmith]
	Comment 2 • 10 years ago

Can we reproduce this on Flame?

	Diego Marcos [:dmarcos]
	Comment 3 • 10 years ago

No. Flame front camera does have face tracking because the HW doesn't support focus regions.

	Diego Marcos [:dmarcos]
	Comment 4 • 10 years ago

(In reply to Diego Marcos [:dmarcos] from comment #3)
> No. Flame front camera does have face tracking because the HW doesn't
> support focus regions.

I meant Flame front DOES NOT support face tracking

	Diego Marcos [:dmarcos]
	Comment 5 • 10 years ago

If any camera support both face tracking + focus regions the feature will be available automatically

	Hema Koka [:hema]
	Comment 6 • 10 years ago

If Face Detection and Focus regions are both available, face tracking will be enabled automatically. Otherwise it is disabled. 

Bhargav: Can you confirm if QRD supports focus regions on front camera?

	Diego Marcos [:dmarcos]
	Comment 7 • 10 years ago

I don't have a QRD device but my guess is that either face tracking or focus regions are not available for the front camera so the feature is not available.

	Diego Marcos [:dmarcos]
	Comment 8 • 10 years ago

On flame we could paint the circles around the faces but we would not be able to focus on them. It would be misleading for the user.

	Lawrence Mandel [:lmandel] (use needinfo)
	Comment 9 • 10 years ago

Clearing the nom on the invalid bug. Comment 6 has justification.

	Inder
	Comment 10 • 10 years ago

Hema -- we confirmed that 8x10 QRD has fixed focus sensor so it won't have face detection circles. We are trying to verify on 8926.

	Pooja
	Comment 11 • 10 years ago

On 8x26 targets the front camera focus mode is fixed and so focus region in not supported.