From fcbbb31f27f00efe7314031bac23e1e011954ad6 Mon Sep 17 00:00:00 2001 From: Niclas Kellgren Date: Fri, 14 Jan 2011 09:11:35 +0100 Subject: [PATCH] DO NOT MERGE Added check to make orientation calculations more robust Added check avoid a division by zero resulting in NaN which in turn makes checkFullyTilted to ignore high tilt angles from then on. If (x, y, z) == (0, 0, 0) then there is no tilt or rotation and this vector must be ignored. This check is extended to ignore all small acceleration values where noise can be of big influence. Low or zero readings can happen when space travelling free falling, but more commonly when shaking or getting bad readings from the sensor. The accelerometer is turned off when not used and polling it too soon after it is turned on may result in (0, 0, 0). Change-Id: I7b61b77c8a9d4e137bf6352b84f0c358d93c82ae --- core/java/android/view/WindowOrientationListener.java | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/core/java/android/view/WindowOrientationListener.java b/core/java/android/view/WindowOrientationListener.java index 2a76e33f94fe9..6095a64d634a6 100755 --- a/core/java/android/view/WindowOrientationListener.java +++ b/core/java/android/view/WindowOrientationListener.java @@ -234,6 +234,14 @@ public abstract class WindowOrientationListener { // high time constant. private static final float MAX_DEVIATION_FROM_GRAVITY = 1.5f; + // Minimum acceleration considered, in m/s^2. Below this threshold sensor noise will have + // significant impact on the calculations and in case of the vector (0, 0, 0) there is no + // defined rotation or tilt at all. Low or zero readings can happen when space travelling + // or free falling, but more commonly when shaking or getting bad readings from the sensor. + // The accelerometer is turned off when not used and polling it too soon after it is + // turned on may result in (0, 0, 0). + private static final float MIN_ABS_ACCELERATION = 1.5f; + // Actual sampling period corresponding to SensorManager.SENSOR_DELAY_NORMAL. There's no // way to get this information from SensorManager. // Note the actual period is generally 3-30ms larger than this depending on the device, but @@ -347,6 +355,9 @@ public abstract class WindowOrientationListener { float deviation = Math.abs(magnitude - SensorManager.STANDARD_GRAVITY); handleAccelerationDistrust(deviation); + if (magnitude < MIN_ABS_ACCELERATION) { + return; // Ignore tilt and orientation when (0, 0, 0) or low reading + } // only filter tilt when we're accelerating float alpha = 1;