# HG changeset patch
# User Ideenmodellierer
# Date 1616617721 -3600
# Node ID c737cf5d906774576e907bbaff99ddde13f7fcd9
# Parent  ed5422ac4ffbe5882db184dbe712ee24688e731c
Do not show focus indicator in case motion detection is suspended:
Motion detection will be suspended in case the diver is operating the OSTC using the buttons. In previous versions the focus indicator was shown even the motion detection was not active => may cause confusion. The indicator is now only shown in case the detection is active

Added hystresis for switching between sector views:
In previous versions small jitter at the angle signal could cause fast switching of the custom view. An offset has now been added to avoid this scenario

diff -r ed5422ac4ffb -r c737cf5d9067 Discovery/Inc/motion.h
--- a/Discovery/Inc/motion.h	Thu Mar 11 21:45:22 2021 +0100
+++ b/Discovery/Inc/motion.h	Wed Mar 24 21:28:41 2021 +0100
@@ -92,6 +92,7 @@
 void calibrateViewport(float roll, float pitch, float yaw);
 float checkViewport(float roll, float pitch, float yaw, uint8_t enableAxis);
 uint8_t viewInFocus(void);
+uint8_t viewDetectionSuspended(void);
 void resetFocusState(void);
 
 #endif /* INC_MOTION_H_ */
diff -r ed5422ac4ffb -r c737cf5d9067 Discovery/Src/base.c
--- a/Discovery/Src/base.c	Thu Mar 11 21:45:22 2021 +0100
+++ b/Discovery/Src/base.c	Wed Mar 24 21:28:41 2021 +0100
@@ -522,7 +522,7 @@
         	{
         		if(get_globalState() != StD)
         		{
-        			suspendMotionDetection(1);		/* do not change custom views while divers is operating the computer */
+        			suspendMotionDetection(10);		/* do not change custom views while divers is operating the computer */
         		}
         		HandleMotionDetection();
         	}
@@ -790,7 +790,7 @@
 
 						if ((status.page == PageDive) && (status.line == 0))
 						{
-							if((action == ACTION_BUTTON_ENTER) && ((pSettings->MotionDetection == MOTION_DETECT_SECTOR) || (pSettings->MotionDetection == MOTION_DETECT_SCROLL)))
+							if(action == ACTION_BUTTON_ENTER)
 							{
 								suspendMotionDetection(10);
 							}
@@ -799,7 +799,9 @@
 						else if (status.page == PageSurface)
 							tHome_change_customview_button_pressed(action);
 						else
+						{
 							tHomeDiveMenuControl(action);
+						}
 					}
 			break;
 
diff -r ed5422ac4ffb -r c737cf5d9067 Discovery/Src/motion.c
--- a/Discovery/Src/motion.c	Thu Mar 11 21:45:22 2021 +0100
+++ b/Discovery/Src/motion.c	Wed Mar 24 21:28:41 2021 +0100
@@ -277,19 +277,26 @@
 /* Map the current pitch value to a sector and create button event in case the sector is left */
 detectionState_t detectSectorButtonEvent(float focusOffset)
 {
+	static uint8_t lastTargetSector = 0xFF;
+	static float lastfocusOffset = 0.0;
+
 	uint8_t newTargetSector;
 
 	newTargetSector = GetSectorForFocus(focusOffset);
 
-	if(settingsGetPointer()->design == 3)		/* Big font view ? */
+	/* take a small hysteresis into account to avoid fast display changes (flicker) */
+	if((newTargetSector != lastTargetSector) && (fabsf(focusOffset - lastfocusOffset) > (sectorDetection.size / 3)))
 	{
-		t3_select_customview(GetCVForSector(newTargetSector));
+		lastfocusOffset = focusOffset;
+		if(settingsGetPointer()->design == 3)		/* Big font view ? */
+		{
+			t3_select_customview(GetCVForSector(newTargetSector));
+		}
+		else
+		{
+			t7_select_customview(GetCVForSector(newTargetSector));
+		}
 	}
-	else
-	{
-		t7_select_customview(GetCVForSector(newTargetSector));
-	}
-
 	return DETECT_NOTHING;
 }
 
@@ -734,6 +741,17 @@
 	inFocus = 0;
 }
 
+uint8_t viewDetectionSuspended(void)
+{
+	uint8_t retVal = 0;
+
+	if(suspendMotionDetectionSec)
+	{
+		retVal = 1;
+	}
+	return retVal;
+}
+
 void suspendMotionDetection(uint8_t seconds)
 {
 	suspendMotionDetectionSec = seconds * 10;		/* detection function is called every 100ms */
diff -r ed5422ac4ffb -r c737cf5d9067 Discovery/Src/t3.c
--- a/Discovery/Src/t3.c	Thu Mar 11 21:45:22 2021 +0100
+++ b/Discovery/Src/t3.c	Wed Mar 24 21:28:41 2021 +0100
@@ -415,7 +415,7 @@
     start.x = 0;
     stop.x = 799;
     stop.y = start.y = BigFontSeperationTopBottom;
-    if(viewInFocus())
+    if((viewInFocus()) && (!viewDetectionSuspended()))
     {
     	GFX_draw_line(tXscreen, start, stop, CLUT_Font023);
     }
@@ -429,7 +429,7 @@
     stop.y = 479;
 
     stop.x = start.x = BigFontSeperationLeftRight;
-    if(viewInFocus())
+    if((viewInFocus() && (!viewDetectionSuspended())))
     {
     	GFX_draw_line(tXscreen, start, stop, CLUT_Font023);
     }
diff -r ed5422ac4ffb -r c737cf5d9067 Discovery/Src/t7.c
--- a/Discovery/Src/t7.c	Thu Mar 11 21:45:22 2021 +0100
+++ b/Discovery/Src/t7.c	Wed Mar 24 21:28:41 2021 +0100
@@ -2935,7 +2935,7 @@
     WidthHeight.x = CUSTOMBOX_LINE_RIGHT - CUSTOMBOX_LINE_LEFT;
     LeftLow.y = 60;
     WidthHeight.y = 440 - LeftLow.y;
-    if(viewInFocus())
+    if((viewInFocus() && (!viewDetectionSuspended())))
     {
     	GFX_draw_box(&t7screen, LeftLow, WidthHeight, 1, CLUT_Font023);
     }