Merge branch 'espruino:master' into master

apps/bthrm/ChangeLog

@@ -20,3 +20,4 @@
 0.07: Recorder icon only blue if values actually arive
       Adds some preset modes and a custom one
       Restructure the settings menu
+0.08: Allow scanning for devices in settings

apps/bthrm/boot.js

@@ -23,7 +23,10 @@
     }
 
     function getCache(){
-      return require('Storage').readJSON("bthrm.cache.json", true) || {};
+      var cache = require('Storage').readJSON("bthrm.cache.json", true) || {};
+      if (settings.btname && settings.btname == cache.name) return cache;
+      clearCache();
+      return {};
     }
     
     function addNotificationHandler(characteristic){
@@ -361,7 +364,13 @@
       var promise;
       
       if (!device){
-        promise = NRF.requestDevice({ filters: serviceFilters });
+        var filters = serviceFilters;
+        if (settings.btname){
+          log("Configured device name", settings.btname);
+          filters = [{name: settings.btname}];
+        }
+        log("Requesting device with filters", filters);
+        promise = NRF.requestDevice({ filters: filters });
         
         if (settings.gracePeriodRequest){
           log("Add " + settings.gracePeriodRequest + "ms grace period after request");
@@ -488,11 +497,15 @@
         if (gatt) {
           if (gatt.connected){
             log("Disconnect with gatt: ", gatt);
+            try{
               gatt.disconnect().then(()=>{
                 log("Successful disconnect");
               }).catch((e)=>{
-              log("Error during disconnect", e);
+                log("Error during disconnect promise", e);
               });
+            } catch (e){
+              log("Error during disconnect attempt", e);
+            }
           }
         }
       }

apps/bthrm/metadata.json

@@ -2,7 +2,7 @@
   "id": "bthrm",
   "name": "Bluetooth Heart Rate Monitor",
   "shortName": "BT HRM",
-  "version": "0.07",
+  "version": "0.08",
   "description": "Overrides Bangle.js's build in heart rate monitor with an external Bluetooth one.",
   "icon": "app.png",
   "type": "app",

apps/bthrm/settings.js

@@ -17,6 +17,7 @@
   var settings;
   readSettings();
 
+  function buildMainMenu(){
     var mainmenu = {
       '': { 'title': 'Bluetooth HRM' },
       '< Back': back,
@@ -57,14 +58,32 @@
           }
           writeSettings("mode",v);
         }
-    },
+      }
-    'Custom Mode': function() { E.showMenu(submenu_custom); },
-    'Debug': function() { E.showMenu(submenu_debug); }
     };
 
+    if (settings.btname){
+      var name = "Clear " + settings.btname;
+      mainmenu[name] = function() {
+      E.showPrompt("Clear current device name?").then((r)=>{
+          if (r) {
+            writeSettings("btname",undefined);
+          }
+          E.showMenu(buildMainMenu());
+        });
+      };
+    }
+
+    mainmenu["BLE Scan"] = ()=> createMenuFromScan();
+    mainmenu["Custom Mode"] = function() { E.showMenu(submenu_custom); };
+    mainmenu.Debug = function() { E.showMenu(submenu_debug); };
+    return mainmenu;
+  }
+  
+
+  
   var submenu_debug = {
     '' : { title: "Debug"},
-    '< Back': function() { E.showMenu(mainmenu); },
+    '< Back': function() { E.showMenu(buildMainMenu()); },
     'Alert on disconnect': {
       value: !!settings.warnDisconnect,
       format: v => settings.warnDisconnect ? "On" : "Off",
@@ -82,9 +101,40 @@
     'Grace periods': function() { E.showMenu(submenu_grace); }
   };
 
+  function createMenuFromScan(){
+    E.showMenu();
+    E.showMessage("Scanning");
+
+    var submenu_scan = {
+      '' : { title: "Scan"},
+      '< Back': function() { E.showMenu(buildMainMenu()); }
+    };
+    var packets=10;
+    var scanStart=Date.now();
+    NRF.setScan(function(d) {
+      packets--;
+      if (packets<=0 || Date.now() - scanStart > 5000){
+        NRF.setScan();
+        E.showMenu(submenu_scan);
+      } else if (d.name){
+        print("Found device", d);
+        submenu_scan[d.name] = function(){
+          E.showPrompt("Set "+d.name+"?").then((r)=>{
+          if (r) {
+            writeSettings("btname",d.name);
+          }
+          E.showMenu(buildMainMenu());
+        });
+        };
+      }
+    }, { filters: [{services: [ "180d" ]}]});
+  }
+  
+
+  
   var submenu_custom = {
     '' : { title: "Custom mode"},
-    '< Back': function() { E.showMenu(mainmenu); },
+    '< Back': function() { E.showMenu(buildMainMenu()); },
     'Replace HRM': {
       value: !!settings.custom_replace,
       format: v => settings.custom_replace ? "On" : "Off",
@@ -165,7 +215,7 @@
   
   var submenu = {
     '' : { title: "Grace periods"},
-    '< Back': function() { E.showMenu(mainmenu); },
+    '< Back': function() { E.showMenu(buildMainMenu()); },
     'Request': {
       value: settings.gracePeriodRequest,
       min: 0,
@@ -208,5 +258,5 @@
     }
   };
   
-  E.showMenu(mainmenu);
+  E.showMenu(buildMainMenu());
 })

apps/widbaroalarm/ChangeLog

@@ -1 +1,2 @@
 0.01: Initial version
+0.02: Do not warn multiple times for the same exceedance

apps/widbaroalarm/README.md

@@ -2,10 +2,8 @@
 
 Get a notification when the pressure reaches defined thresholds.
 
-![Screenshot](screenshot.png)
 
 ## Settings
-
 * Interval: check interval of sensor data in minutes. 0 to disable automatic check.
 * Low alarm: Toggle low alarm
   * Low threshold: Warn when pressure drops below this value

apps/widbaroalarm/metadata.json

@@ -1,8 +1,8 @@
 {
   "id": "widbaroalarm",
-  "name": "Barometer alarm widget",
+  "name": "Barometer Alarm Widget",
-  "shortName": "Barometer alarm",
+  "shortName": "Barometer Alarm",
-  "version": "0.01",
+  "version": "0.02",
   "description": "A widget that can alarm on when the pressure reaches defined thresholds.",
   "icon": "widget.png",
   "type": "widget",

apps/widbaroalarm/settings.js

@@ -36,7 +36,7 @@
         value: settings.min,
         min: 600,
         max: 1000,
-        step: 10,
+        step: 5,
         onchange: x => save("min", x),
       },
       "High alarm": {
@@ -48,9 +48,9 @@
       },
       "High threshold": {
         value: settings.max,
-        min: 1000,
+        min: 700,
         max: 1100,
-        step: 10,
+        step: 5,
         onchange: x => save("max", x),
       },
       "Drop alarm": {

apps/widbaroalarm/widget.js

@@ -6,14 +6,28 @@
   const LOG_FILE = "widbaroalarm.log.json";
   const SETTINGS_FILE = "widbaroalarm.json";
   const storage = require('Storage');
-    let settings = Object.assign(
+
+  let settings;
+
+  function loadSettings() {
+    settings = Object.assign(
       storage.readJSON("widbaroalarm.default.json", true) || {},
       storage.readJSON(SETTINGS_FILE, true) || {}
     );
+  }
+
+  loadSettings();
+
 
   function setting(key) {
     return settings[key];
   }
+
+  function saveSetting(key, value) {
+    settings[key] = value;
+    storage.write(SETTINGS_FILE, settings);
+  }
+
   const interval = setting("interval");
 
   let history3 = storage.readJSON(LOG_FILE, true) || []; // history of recent 3 hours
@@ -33,11 +47,16 @@
     }
   }
 
-    let alreadyWarned = false;
+
+  function didWeAlreadyWarn(key) {
+    return setting(key) == undefined || setting(key) > 0;
+  }
 
   function checkForAlarms(pressure) {
     if (pressure == undefined || pressure <= 0) return;
 
+    let alreadyWarned = false;
+
     const ts = Math.round(Date.now() / 1000); // seconds
     const d = {
       "ts": ts,
@@ -55,18 +74,35 @@
       history3.shift();
     }
 
-      history3.push(d);
+    if (setting("lowalarm")) {
-      // write data to storage
+      // Is below the alarm threshold?
-      storage.writeJSON(LOG_FILE, history3);
+      if (pressure <= setting("min")) {
-
+        if (!didWeAlreadyWarn("lastLowWarningTs")) {
-      if (setting("lowalarm") && pressure <= setting("min")) {
           showAlarm("Pressure low: " + Math.round(pressure) + " hPa");
+          saveSetting("lastLowWarningTs", ts);
           alreadyWarned = true;
         }
-      if (setting("highalarm") && pressure >= setting("max")) {
+      } else {
+        saveSetting("lastLowWarningTs", 0);
+      }
+    } else {
+      saveSetting("lastLowWarningTs", 0);
+    }
+
+    if (setting("highalarm")) {
+      // Is above the alarm threshold?
+      if (pressure >= setting("max")) {
+        if (!didWeAlreadyWarn("lastHighWarningTs")) {
           showAlarm("Pressure high: " + Math.round(pressure) + " hPa");
+          saveSetting("lastHighWarningTs", ts);
           alreadyWarned = true;
         }
+      } else {
+        saveSetting("lastHighWarningTs", 0);
+      }
+    } else {
+      saveSetting("lastHighWarningTs", 0);
+    }
 
     if (!alreadyWarned) {
       // 3h change detection
@@ -86,21 +122,39 @@
           // drop alarm
           if (drop3halarm > 0 && oldestPressure > pressure) {
             if (Math.abs(diff) > drop3halarm) {
+              if (!didWeAlreadyWarn("lastDropWarningTs")) {
                 showAlarm((Math.round(Math.abs(diff) * 10) / 10) + " hPa/3h from " +
                   Math.round(oldestPressure) + " to " + Math.round(pressure) + " hPa", "Pressure drop");
+                saveSetting("lastDropWarningTs", ts);
               }
+            } else {
+              saveSetting("lastDropWarningTs", 0);
+            }
+          } else {
+            saveSetting("lastDropWarningTs", 0);
           }
 
           // raise alarm
           if (raise3halarm > 0 && oldestPressure < pressure) {
             if (Math.abs(diff) > raise3halarm) {
+              if (!didWeAlreadyWarn("lastRaiseWarningTs")) {
                 showAlarm((Math.round(Math.abs(diff) * 10) / 10) + " hPa/3h from " +
                   Math.round(oldestPressure) + " to " + Math.round(pressure) + " hPa", "Pressure raise");
+                saveSetting("lastRaiseWarningTs", ts);
+              }
+            } else {
+              saveSetting("lastRaiseWarningTs", 0);
+            }
+          } else {
+            saveSetting("lastRaiseWarningTs", 0);
           }
         }
       }
     }
-  }
+
+    history3.push(d);
+    // write data to storage
+    storage.writeJSON(LOG_FILE, history3);
 
     // calculate 3h average for widget
     let sum = 0;
@@ -155,6 +209,15 @@ function reload() {
   }
 
   function draw() {
+    if (global.WIDGETS != undefined && typeof WIDGETS === "object") {
+      WIDGETS["baroalarm"] = {
+        width: setting("show") ? 24 : 0,
+        reload: reload,
+        area: "tr",
+        draw: draw
+      };
+    }
+
     g.reset();
     if (setting("show") && medianPressure != undefined) {
       g.setFont("6x8", 1).setFontAlign(1, 0);
@@ -165,21 +228,12 @@ function draw() {
     }
   }
 
-if (global.WIDGETS != undefined && typeof WIDGETS === "object") {
-  WIDGETS["baroalarm"] = {
-    width: setting("show") ? 24 : 0,
-    reload: reload,
-    area: "tr",
-    draw: draw
-  };
-}
-
   // Let's delay the first check a bit
   setTimeout(function() {
     check();
     if (interval > 0) {
       setInterval(check, interval * 60000);
     }
-}, 5000);
+  }, 1000);
 
 })();

core

@@ -1 +1 @@
-Subproject commit a7a80a13fa187a4ff5f89669992babca2d95812c
+Subproject commit affb0b15b41eb35a1548373831af7001bad64435