Merge branch 'master' of github.com:espruino/BangleApps

apps/game1024/ChangeLog

@@ -3,4 +3,5 @@
 0.03: Basic colors
 0.04: Bug fix score reset after Game Over, new icon
 0.05: Chevron marker on the randomly added square
-0.06: Fixed issue 1609 added a message popup state handler to control unwanted screen redraw
+0.06: Fixed issue 1609 added a message popup state handler to control unwanted screen redraw
+0.07: Optimized the mover algorithm for efficiency (work in progress)

apps/game1024/app.js

@@ -164,16 +164,16 @@ const buttons = {
 const mover = {
   direction: {
     up: {name: 'up', step: 1, innerBegin: 0,    innerEnd: rows-1, outerBegin: 0,    outerEnd: cols-1, iter: rows -1,
-      sqIndex: function (m,n) {return m*(cols) + n;}, sqNextIndex: function (m,n) {return m < rows -1 ? (m+1)*(cols) + n : -1;}
+      sqIndex: function (i,o) {return i*(cols) + o;}, sqNextIndex: function (i,o) {return i < rows -1 ? (i+1)*(cols) + o : -1;}
     }, 
     down: {name: 'down', step:-1, innerBegin: rows-1, innerEnd: 0,    outerBegin: cols-1, outerEnd: 0, iter: rows -1,
-      sqIndex: function (m,n) {return m*(cols) + n;}, sqNextIndex: function (m,n) {return m > 0 ? (m-1)*(cols) + n : -1;}
+      sqIndex: function (i,o) {return i*(cols) + o;}, sqNextIndex: function (i,o) {return i > 0 ? (i-1)*(cols) + o : -1;}
     }, 
     left: {name: 'left', step: 1, innerBegin: 0,    innerEnd: cols-1, outerBegin: 0,    outerEnd: rows-1, iter: cols -1,
-      sqIndex: function (m,n) {return n*(rows) + m;}, sqNextIndex: function (m,n) {return m < cols -1 ? n*(rows) + m +1 : -1;}
+      sqIndex: function (i,o) {return o*(rows) + i;}, sqNextIndex: function (i,o) {return i < cols -1 ? o*(rows) + i +1 : -1;}
     },
     right: {name: 'right', step:-1, innerBegin: cols-1, innerEnd: 0,    outerBegin: rows-1, outerEnd: 0, iter: cols -1,
-      sqIndex: function (m,n) {return n*(rows) + m;}, sqNextIndex: function (m,n) {return m > 0 ? n*(rows) + m -1: -1;}
+      sqIndex: function (i,o) {return o*(rows) + i;}, sqNextIndex: function (i,o) {return i > 0 ? o*(rows) + i -1: -1;}
     }
   },
   anyLeft: function() {
@@ -207,49 +207,39 @@ const mover = {
     });
     return canContinue;
   },
-  nonEmptyCells: function (dir) {
-    debug(() => console.log("Move: ", dir.name));
+  moveAndMerge: function (dir) {
     const step = dir.step;
     // outer loop for all colums/rows
-    for (let n = dir.outerBegin; step*n <= step*dir.outerEnd; n=n+step) {
-      // let rowStr = '| ';
-
-      // Move a number of iteration with the squares to move them all to one side
-      for (let iter = 0; iter < dir.iter; iter++) {
-
-        // lets move squares one position in a row or column, counting backwards starting from the and where the squares will end up
-        for (let m = dir.innerBegin; step*m <= step*dir.innerEnd; m=m+step) {
-          // get the array of squares index for current cell
-          const idx = dir.sqIndex(m,n);
-          const nextIdx = dir.sqNextIndex(m,n);
-
-          if (allSquares[idx].expVal == 0 && nextIdx >= 0) {
-            allSquares[idx].setExpVal(allSquares[nextIdx].expVal);
-            allSquares[nextIdx].setExpVal(0);
-          }
-        }
-      }
-    }
-  },
-  // add up the conjacent squares with identical values en set next square to empty in the process
-  mergeEqlCells: function(dir) {
-    const step = dir.step;
-    // outer loop for all colums/rows
-    for (let n = dir.outerBegin; step*n <= step*dir.outerEnd; n=n+step) {
-      // lets move squares one position in a row or column, counting backwards starting from the and where the squares will end up
+    for (let o = dir.outerBegin; step*o <= step*dir.outerEnd; o=o+step) {
+      
+      let allVals = allSquares.map(sq=>{return sq.getExpVal()});
+      let allLineVals = [];
       for (let m = dir.innerBegin; step*m <= step*dir.innerEnd; m=m+step) {
-        const idx = dir.sqIndex(m,n);
-        const nextIdx = dir.sqNextIndex(m,n);
-
-        if ((allSquares[idx].expVal > 0) && nextIdx >= 0) {
-          if (allSquares[idx].expVal == allSquares[nextIdx].expVal) {
-            let expVal = allSquares[idx].expVal;
-            allSquares[idx].setExpVal(++expVal);
-            allSquares[idx].addToScore();
-            allSquares[nextIdx].setExpVal(0);
-          }
-        }
+        allLineVals.push(allVals[dir.sqIndex(m,o)]);
       }
+      
+      let sortedLineVals = allLineVals.filter((val)=>{return val>0;});
+      let zeroLineVals = allLineVals.filter((val)=>{return val==0;});
+      // merge the equally valued adjacent cells
+      let r=0;
+      while (r<sortedLineVals.length-1) {
+        if (sortedLineVals[r] == sortedLineVals[r+1]) {
+          ++sortedLineVals[r];
+          addToScore(sortedLineVals[r]);
+          sortedLineVals[++r] = 0;
+        }
+        r++;
+      }
+      let mergedLineVals = sortedLineVals.filter((val)=>{return val>0;});
+      sortedLineVals.filter((val)=>{return val==0;}).forEach((zero)=>{mergedLineVals.push(zero);});
+      zeroLineVals.forEach((zero)=>{mergedLineVals.push(zero);});
+      
+      let i = 0;
+      for (let m = dir.innerBegin; step*m <= step*dir.innerEnd; m=m+step) {
+        let idx = dir.sqIndex(m,o);
+        allSquares[idx].setExpVal(mergedLineVals[i++]);
+      }
+      debug(()=>console.log("new allSquares values:", allSquares.map(sq=>{return sq.expVal})));
     }
   }
 };
@@ -301,7 +291,7 @@ class Button {
 }
 
 class Cell {
-  constructor(x0, y0, width, idx, cb) {
+  constructor(x0, y0, width, idx) {
     this.x0 = x0;
     this.y0 = y0;
     this.x1 = x0 + width;
@@ -309,7 +299,7 @@ class Cell {
     this.expVal = 0;
     this.previousExpVals=[];
     this.idx = idx;
-    this.cb = cb;
+    // this.cb = cb;
     this.isRndm = false;
     this.ax = x0;
     this.ay = Math.floor(0.2*width+y0);
@@ -345,6 +335,9 @@ class Cell {
        .drawString(char, strX, strY);
     }
   }
+  getExpVal() {
+    return this.expVal;
+  }
   setExpVal(val) {
     this.expVal = val;
   }
@@ -364,10 +357,6 @@ class Cell {
   removeUndo() {
     this.previousExpVals=[0];
   }
-  addToScore() {if (typeof this.cb === 'function') {
-      this.cb(this.expVal);
-    }
-  }
   setRndmFalse() {
     this.isRndm = false;
   }
@@ -416,7 +405,7 @@ function createGrid () {
     for (let c = 0; c < cols; c++) {
       let x0 = borderWidth + c*(borderWidth + sqWidth) - (rows/2)*(2*borderWidth + sqWidth) + middle.x + Math.floor(sqWidth/3);
       let y0 = borderWidth + r*(borderWidth + sqWidth) - (cols/2)*(2*borderWidth + sqWidth) + middle.y + Math.floor(sqWidth/3);
-      let cell = new Cell(x0, y0, sqWidth, c + r*cols, addToScore);
+      let cell = new Cell(x0, y0, sqWidth, c + r*cols);
       allSquares.push(cell);
     }
   }
@@ -454,7 +443,7 @@ function addRandomNumber() {
     let randomIdx = Math.floor( emptySquaresIdxs.length * Math.random() );
     allSquares[emptySquaresIdxs[randomIdx]].setExpVal(makeRandomNumber());
     allSquares[emptySquaresIdxs[randomIdx]].setRndmTrue();
-  } 
+  }
 }
 function drawGrid() {
   allSquares.forEach(sq => {
@@ -651,9 +640,7 @@ dragger.attach();
 function runGame(dir){
   addToUndo();
   updUndoLvlIndex();
-  mover.nonEmptyCells(dir);
-  mover.mergeEqlCells(dir);
-  mover.nonEmptyCells(dir);
+  mover.moveAndMerge(dir);
   allSquares.forEach(sq => {sq.setRndmFalse();});
   addRandomNumber();
   drawGrid();

apps/game1024/metadata.json

@@ -1,7 +1,7 @@
 { "id": "game1024",
     "name": "1024 Game",
     "shortName" : "1024 Game",
-    "version": "0.06",
+    "version": "0.07",
     "icon": "game1024.png",
     "screenshots": [ {"url":"screenshot.png" } ],
     "readme":"README.md",

apps/timerclk/ChangeLog

@@ -1 +1,2 @@
 0.01: New App!
+0.02: Add sunrise/sunset. Fix timer bugs.

apps/timerclk/README.md

@@ -11,6 +11,7 @@ A clock based on the Anton Clock with stopwatches, timers and alarms based on th
 * alarms
 * multiple stopwatches, timers and alarms
 * stopwatches and timers keep running in the background
+* optional time of sunrise/sunset using the My Location app - hidden by default
 
 ## Images
 

apps/timerclk/alarm.info

@@ -1 +0,0 @@
-{"id":"timerclk","name":"tclk Alarm","src":"timerclk.alarm.js","icon":"timerclk.img","version":"0.01","tags":"","files":"","sortorder":10}

apps/timerclk/boot.js

@@ -1,26 +1,17 @@
 var timerclkTimerTimeout;
 var timerclkAlarmTimeout;
 function timerclkCheckTimers() {
+  var expiresIn=require("timerclk.lib.js").timerExpiresIn;
   if (timerclkTimerTimeout) clearTimeout(timerclkTimerTimeout);
   var timers = require('Storage').readJSON('timerclk.timer.json',1)||[];
   timers = timers.filter(e=>e.start);
   if (timers.length) {
-    timers = timers.sort((a,b)=>{
-      var at = a.timeAdd;
-      if (a.start) at += Date.now()-a.start;
-      at = a.period-at;
-      var bt = b.timeAdd;
-      if (b.start) bt += Date.now()-b.start;
-      bt = b.period-bt;
-      return at-bt;
-    });
+    timers = timers.sort((a,b)=>expiresIn(a)-expiresIn(b));
     if (!require('Storage').read("timerclk.timer.alert.js")) {
       console.log("No timer app!");
     } else {
-      var time = timers[0].timeAdd;
-      if (timers[0].start) time += Date.now()-timers[0].start;
-      time = timers[0].time - time;
-      if (time<1000) t=1000;
+      var time = expiresIn(timers[0]);
+      if (time<1000) time=1000;
       if (timerclkTimerTimeout) clearTimeout(timerclkTimerTimeout);
       timerclkTimerTimeout = setTimeout(() => load("timerclk.timer.alert.js"),time);
     }
@@ -38,7 +29,7 @@ function timerclkCheckAlarms() {
     } else {
       var time = alarms[0].time-currentTime;
       if (alarms[0].last == new Date().getDate() || time < 0) time += 86400000;
-      if (time<1000) t=1000;
+      if (time<1000) time=1000;
       if (timerclkAlarmTimeout) clearTimeout(timerclkAlarmTimeout);
       timerclkAlarmTimeout = setTimeout(() => load("timerclk.alarm.alert.js"),time);
     }

apps/timerclk/lib.js

@@ -125,3 +125,5 @@ exports.registerControls = function(o) {
     });
   }
 };
+
+exports.timerExpiresIn=t=>t.time-(Date.now()-t.start);

apps/timerclk/metadata.json

@@ -2,7 +2,7 @@
 	"id": "timerclk",
 	"name": "Timer Clock",
 	"shortName":"Timer Clock",
-	"version":"0.01",
+	"version":"0.02",
 	"description": "A clock with stopwatches, timers and alarms build in.",
 	"icon": "app-icon.png",
 	"type": "clock",
@@ -28,10 +28,7 @@
 		{"name":"timerclk.timer.js","url":"timer.js"},
 		{"name":"timerclk.timer.alert.js","url":"timer.alert.js"},
 		{"name":"timerclk.alarm.js","url":"alarm.js"},
-		{"name":"timerclk.alarm.alert.js","url":"alarm.alert.js"},
-		{"name":"timerclk.stopwatch.info","url":"stopwatch.info"},
-		{"name":"timerclk.timer.info","url":"timer.info"},
-		{"name":"timerclk.alarm.info","url":"alarm.info"}
+		{"name":"timerclk.alarm.alert.js","url":"alarm.alert.js"}
 	],
 	"data": [{"name":"timerclk.json"},{"name":"timerclk.stopwatch.json"},{"name":"timerclk.timer.json"},{"name":"timerclk.alarm.json"}],
 	"sortorder": 0

apps/timerclk/settings.js

@@ -12,9 +12,12 @@
     "dowFontSize":2,
     "specialFont":"6x8",
     "specialFontSize":2,
+    "srssFont":"6x8",
+    "srssFontSize":2,
     "shortDate":true,
     "showStopwatches":true,
     "showTimers":true,
+    "showSrss":false,
   }, settings.clock||{});
   settings.stopwatch = Object.assign({
     "font":"Vector",
@@ -108,6 +111,23 @@
         writeSettings();
       }
     },
+    "sun font":{
+      value: 0|g.getFonts().indexOf(settings.clock.srssFont),
+      format: v => g.getFonts()[v],
+      min: 0, max: g.getFonts().length-1,
+      onchange: v => {
+        settings.clock.srssFont = g.getFonts()[v];
+        writeSettings();
+      }
+    },
+    "sun size":{
+      value: 0|settings.clock.srssFontSize,
+      min: 0,
+      onchange: v => {
+        settings.clock.srssFontSize = v;
+        writeSettings();
+      }
+    },
     "short date": {
       value: !!settings.clock.shortDate,
       format: BOOL_FORMAT,
@@ -132,6 +152,14 @@
         writeSettings();
       }
     },
+    "sun times": {
+      value: !!settings.clock.showSrss,
+      format: v=>v?/*LANG*/"Show":/*LANG*/"Hide",
+      onchange: v => {
+        settings.clock.showSrss = v;
+        writeSettings();
+      }
+    },
   };
   
   var stopwatchMenu = {

apps/timerclk/stopwatch.info

@@ -1 +0,0 @@
-{"id":"timerclk","name":"tclk Stopwatch","src":"timerclk.stopwatch.js","icon":"timerclk.img","version":"0.01","tags":"","files":"","sortorder":10}

apps/timerclk/timer.alert.js

@@ -14,10 +14,7 @@ function showTimer(timer) {
     buttons : {/*LANG*/"Ok":true}
   }).then(function(ok) {
     buzzCount = 0;
-    if (ok) {
-      timer.time += Date.now() - timer.start;
-      timer.start = null;
-    }
+    timer.start = null;
     require("Storage").write("timerclk.timer.json",JSON.stringify(timers));
     load();
   });
@@ -45,16 +42,8 @@ console.log("checking for timers...");
 var timers = require("Storage").readJSON("timerclk.timer.json",1)||[];
 var active = timers.filter(e=>e.start);
 if (active.length) {
-  // if there's an timer, show it
-  active = active.sort((a,b)=>{
-    var at = a.time;
-    if (a.start) at += Date.now()-a.start;
-    at = a.period-at;
-    var bt = b.time;
-    if (b.start) bt += Date.now()-b.start;
-    bt = b.period-bt;
-    return at-bt;
-  });
+  // if there's an active timer, show it
+  active = active.sort((a,b)=>timerclk.timerExpiresIn(a)-timerclk.timerExpiresIn(b));
   showTimer(active[0]);
 } else {
   // otherwise just go back to default app

apps/timerclk/timer.info

@@ -1 +0,0 @@
-{"id":"timerclk","name":"tclk Timer","src":"timerclk.timer.js","icon":"timerclk.img","version":"0.01","tags":"","files":"","sortorder":10}

apps/timerclk/timer.js

@@ -34,18 +34,20 @@ function update() {
 }
 function play() {
   if (all[current].start) { // running
-    all[current].timeAdd += Date.now() - all[current].start;
+    all[current].timeAdd = Date.now() - all[current].start;
     all[current].start = null;
     update();
   } else { // paused
-    all[current].start = Date.now();
+    all[current].start = Date.now() - all[current].timeAdd;
+    all[current].timeAdd = 0;
     update();
   }
   require("Storage").write("timerclk.timer.json",JSON.stringify(all));
   timerclkCheckTimers();
 }
 function reset() {
-  all[current] = defaultElement.clone();
+  all[current].start = null;
+  all[current].timeAdd = 0;
   update();
   require("Storage").write("timerclk.timer.json",JSON.stringify(all));
   timerclkCheckTimers();