@uncaught/gpio-shutter-bridge 1.2.1 → 1.2.2

package/dist/Shutter/VeluxShutter.d.ts

@@ -1,9 +1,9 @@
 import { Gpio } from 'onoff';
 import { ShutterState, ShutterInterfaceWithState, ShutterInterfaceWithPosition, ShutterPosition } from './Shutter.js';
 interface Durations {
+    bottomSignalDurations?: number[];
     topFullCloseDurations?: number[];
     topFullOpenDurations?: number[];
-    topSignalDurations?: number[];
 }
 export interface Persistence extends Durations {
     position?: number;

package/dist/Shutter/VeluxShutter.js

@@ -2,6 +2,14 @@ import { Gpio } from 'onoff';
 import { isShutterPosition, isShutterAction, } from './Shutter.js';
 import { press } from '../Gpio.js';
 const durationsToKeep = 20;
+const getBottomDurations = (d) => d.sort((a, b) => a - b).slice(0, durationsToKeep);
+const getLastDurations = (d) => d.slice(-durationsToKeep);
+const getTopDurations = (d) => d.sort((a, b) => b - a).slice(0, durationsToKeep);
+const durationHandlers = {
+    bottomSignalDurations: getBottomDurations,
+    topFullCloseDurations: getTopDurations,
+    topFullOpenDurations: getTopDurations,
+};
 function minMaxPercentage(num) {
     return Math.min(Math.max(num, 0), 100);
 }
@@ -74,10 +82,7 @@ export class VeluxShutter {
     }
     storeDuration(key, duration) {
         if (duration > 0) {
-            //Keep the N longest durations:
-            this.store.set({
-                [key]: [...(this.store.get()[key] ?? []), duration].sort((a, b) => b - a).slice(0, durationsToKeep),
-            });
+            this.store.set({ [key]: durationHandlers[key]([...(this.store.get()[key] ?? []), duration]) });
         }
     }
     getAverageDuration(key) {
@@ -89,7 +94,7 @@ export class VeluxShutter {
         return 0;
     }
     getEstimatedActionDuration(action) {
-        const signalDuration = this.getAverageDuration('topSignalDurations');
+        const signalDuration = this.getAverageDuration('bottomSignalDurations');
         let avg = 0;
         if (action === 'opening') {
             avg = this.getAverageDuration('topFullOpenDurations');
@@ -120,7 +125,7 @@ export class VeluxShutter {
             //Measure the time it takes between a manual "stop" and the following stopped signal.
             // This gives us an estimate on how long the KLF 150 takes to send it for any operation.
             const stopDuration = this.lastStoppingStartTime ? Date.now() - this.lastStoppingStartTime : 0;
-            this.storeDuration('topSignalDurations', stopDuration);
+            this.storeDuration('bottomSignalDurations', stopDuration);
         }
         else {
             this.lastStoppingStartTime = 0;
@@ -133,7 +138,7 @@ export class VeluxShutter {
             }
         }
         if (isShutterPosition(state)) {
-            const signalDuration = this.getAverageDuration('topSignalDurations');
+            const signalDuration = this.getAverageDuration('bottomSignalDurations');
             const measuredDuration = this.lastActionStartTime ? Date.now() - this.lastActionStartTime : 0;
             const positionDuration = measuredDuration > 0 ? Math.max(0, measuredDuration - signalDuration) : 0;
             if (state === 'closed') {

package/package.json

@@ -2,7 +2,7 @@
   "name": "@uncaught/gpio-shutter-bridge",
   "author": "uncaught <uncaught42@gmail.com>",
   "license": "MIT",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "description": "MQTT shutter bridge for home assistant with Velux KLF 150 support",
   "repository": {
     "type": "git",