added callbacks for face detection result in activity #13
Conversation
| @@ -46,6 +47,7 @@ class FaceBoundsOverlay @JvmOverloads constructor(ctx: Context, attrs: Attribute | |||
| canvas.drawId(faceBounds.id.toString(), faceBounds.box.center()) | |||
| canvas.drawBounds(faceBounds.box) | |||
| } | |||
| Log.e("faces", "draw ${facesBounds.size}") | |||
There was a problem hiding this comment.
Adding a log message here makes sense, but instead of an error log message, it should be an info or debug message, so Log.d or Log.i. Also, could you add a static log tag to the class, something like:
companion object {
private const val TAG = "FaceBoundsOverlay"
}Then use it for the logging.
Log.d(TAG, "Drawing ${facesBounds.size}")| mainExecutor.execute { faceBoundsOverlay.updateFaces(faceBounds) } | ||
| mainExecutor.execute { | ||
| faceBoundsOverlay.updateFaces(faceBounds) | ||
| onFaceDetectionResultListener?.onSuccess(faceBounds) |
|
|
||
| override fun onSuccess(faceBounds: List<FaceBounds>) { | ||
| super.onSuccess(faceBounds) | ||
| Log.e(TAG,"total faces ${faceBounds.size}") |
There was a problem hiding this comment.
This isn't an error, it shouldn't use Log.e, instead an info or debug log should be used.
Also, please make sure to format the file after you're done (a space is missing before the log message).
| super.onSuccess(faceBounds) | ||
| Log.e(TAG,"total faces ${faceBounds.size}") | ||
| for (face in faceBounds) { | ||
| Log.d(TAG, "face ${face.id} ${face.box.width()} ${face.box.height()}") |
There was a problem hiding this comment.
One little change here, could you format the log message to be
face ${face.id} [${face.box.width()}, ${face.box.height()}]| override fun onCreate(savedInstanceState: Bundle?) { | ||
| super.onCreate(savedInstanceState) | ||
| setContentView(R.layout.activity_main) | ||
|
|
||
| val lensFacing = | ||
| savedInstanceState?.getSerializable(KEY_LENS_FACING) as Facing? ?: Facing.BACK | ||
| val lensFacing = Facing.FRONT |
There was a problem hiding this comment.
lensFacing is saved in onSaveInstanceState(). Please revert setting it here to use the saved instance state.
| viewfinder.facing = lensFacing | ||
| viewfinder.rotation = 180f |
There was a problem hiding this comment.
Why are you rotating the viewfinder?
| viewfinder.addFrameProcessor { | ||
| faceDetector.process( | ||
| Frame( | ||
| data = it.data, | ||
| rotation = it.rotation, | ||
| size = Size(it.size.width, it.size.height), | ||
| format = it.format, | ||
| lensFacing = if (viewfinder.facing == Facing.BACK) LensFacing.BACK else LensFacing.FRONT | ||
| lensFacing = LensFacing.FRONT |
There was a problem hiding this comment.
Could you set the value of the frame's lensFacing parameter to the variable lensFacing?
| ) | ||
| ) | ||
| } | ||
|
|
||
| toggleCameraButton.setOnClickListener { |
There was a problem hiding this comment.
Please revert removing these lines.
| } | ||
|
|
||
| //111.375 338.5481 |
There was a problem hiding this comment.
Please remove this line and the one below it.
while using face detection, developer may need to do something when face detected in camera. hence i added callbacks for face detection process in main activity. if you find this helpful then you can accept this pull request.
thanks.