Repository navigation
UwbRanging: Integrate sensor fusion feature - #479
jameseidson wants to merge 3 commits into
Conversation
agammann
left a comment
There was a problem hiding this comment.
sensorFusionEnabled defaults to false and ARCore is marked optional, but this adds CAMERA to the unconditional PERMISSIONS_REQUIRED list at startup. Users running ordinary UWB ranging now get a camera prompt even if they never enable sensor fusion, and the denial toast says all permissions are required. Could camera permission be requested only when turning on sensor fusion, leaving the baseline UWB path unchanged?
…egration # Conflicts: # gradle/libs.versions.toml
CAMERA is no longer part of the startup permission set, so the baseline UWB ranging flow is unchanged. The settings toggle requests CAMERA when sensor fusion is turned on, and ARCore availability/install is only checked while sensor fusion is enabled and the camera is granted.
|
I rechecked the updated diff: camera permission is now requested when sensor fusion is enabled in settings, rather than in the baseline startup permission list. That addresses the prompt concern I raised. This is a scoped diff review; I did not run the sample or test on a device. AI-assisted review. |
This is a new feature added to
androidx.core.uwb:uwbwhich supports software-based estimation of angle-of-arrival. Integrate this into the UwbRanging samples app to allow manual testing of the feature and provide example usage for apps.