Added check of HRM measurement confidence before sending HRM data, ot…

…herwise it sends the -1 fault value.
OpenSeizureDetector · Dec 17, 2023 · 23b23da · 23b23da
1 parent ca633d4
commit 23b23da
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/apps/openseizure/settings.js b/apps/openseizure/settings.js
@@ -34,6 +34,21 @@
           E.showMenu(osdMenu);
         }
       },
+      'HR_CONF_THRESH': {
+        value: 0|settings.HR_CONF_THRESH,  // 0| converts undefined to 0
+        min: 0,
+        max : 100,
+        step: 5,
+        format: x => {
+          return x + "%";
+        },
+        onchange: v => {
+          settings.HR_CONF_THRESH = v;
+          writeSettings();
+          E.showMenu(osdMenu);
+        }
+      },
+
     };
     E.showMenu(osdMenu);
   })  //(load)
diff --git a/apps/openseizure/widget.js b/apps/openseizure/widget.js
@@ -61,7 +61,8 @@ const CHAR_HR_LOC = 0x2A38; // Official BLE Sensor Location UUID
 	function reload() {
 		settings = Object.assign({
 			ACC_FMT : 0,
-			TEST_MODE : false
+			TEST_MODE : false,
+			HR_CONF_THRESH : 60
 		}, require('Storage').readJSON('openseizure.json',1)||{});
 		saveSettings(settings);
 	}
@@ -209,7 +210,11 @@ function toByteArray(value, numberOfBytes, isSigned) {
 	// Initialise the HRM
 	Bangle.setHRMPower(1);
 	Bangle.on('HRM', function(hrm) { 
-		hrVal = hrm['bpm'];
+		if (hrm.confidence > settings.HR_CONF_THRESH)
+			hrVal = hrm['bpm'];
+		else
+			hrVal = -1;
+		//hrVal = 100*hrm['confidence']  // FIXME - this is just for testing
 	});