android / android/camera-samples

MLKitAnalyser should use cameraExecutor instead of main executor

Open
#533 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Kotlin
Stars
5.5k
Forks
2.4k
PR merge metrics
No merged PRs in 30d

Description

https://github.com/android/camera-samples/blob/f284e1d9c740fbe0db27ae41b07a7bbc01361499/CameraX-MLKit/app/src/main/java/com/example/camerax_mlkit/MainActivity.kt#L71

```
cameraController.setImageAnalysisAnalyzer(
ContextCompat.getMainExecutor(this), // <- cameraExecutor
MlKitAnalyzer(
listOf(barcodeScanner),
COORDINATE_SYSTEM_VIEW_REFERENCED,
ContextCompat.getMainExecutor(this)
) { result: MlKitAnalyzer.Result? ->
```

Contributor guide

Open the contributing guide

Research direction

Open CameraX-MLKit/app/src/main/java/com/example/camerax_mlkit/MainActivity.kt around line 71 and inspect how cameraExecutor is defined and passed to the analyzer setup. Confirm the intended executor arguments, then verify that the CameraX-MLKit sample still builds and uses cameraExecutor for the requested analysis work.

Written by the indexing model from the issue text.

Assessment

Tech stack
kotlin
Domain
mobile-dev
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.