node-red-contrib-energymeterplus 0.2.3 → 0.2.5

package/README.md

@@ -1,4 +1,4 @@
-#### **EnergyMeterPlus Node Version 0.2.3**
+#### **EnergyMeterPlus Node Version 0.2.5**
 
 
 
@@ -168,7 +168,11 @@ Costs scale automatically with your configured unit cost.
 
 
 
-##### **Updates 2.3:**
+##### **Updates:**
+
+
+
+###### **V2.3:**
 
 
 
@@ -186,3 +190,27 @@ Baselines now applied once and stored in context.
 
 updated counters to continuous absorption, no transfer at rollover
 
+
+
+###### v2.4
+
+
+
+Fixed a bug where the internal counters were not updating incrementally.
+
+
+
+Added payload validation
+
+
+
+Fixed Baselines bug: Baselines captured once, stored in context, and survive after UI clears
+
+
+
+v2.5
+
+
+
+Fixed internal counter bug
+

package/energyMeterPlus.js

@@ -11,13 +11,7 @@ module.exports = function(RED) {
         let inputUnit = config.inputUnit || "W";
         let currencyCode = config.currency || "USD";
 
-        // Baseline corrections (numeric values from config)
-        let baselineDaily   = Number(config.baselineDaily)   || 0;
-        let baselineWeekly  = Number(config.baselineWeekly)  || 0;
-        let baselineMonthly = Number(config.baselineMonthly) || 0;
-        let baselineYearly  = Number(config.baselineYearly)  || 0;
-
-        // Load persisted baseline or initialize
+        // Counters always start from context or zero
         let baseline = node.context().get("baseline") || {
             daily: 0,
             weekly: 0,
@@ -25,43 +19,28 @@ module.exports = function(RED) {
             yearly: 0
         };
 
-        // Store numeric baselines separately so resets don’t pull from blanked config
+        // Capture baselines once on deploy
         let storedBaselines = node.context().get("storedBaselines") || {
-            daily: baselineDaily,
-            weekly: baselineWeekly,
-            monthly: baselineMonthly,
-            yearly: baselineYearly
+            daily: Number(config.baselineDaily) || 0,
+            weekly: Number(config.baselineWeekly) || 0,
+            monthly: Number(config.baselineMonthly) || 0,
+            yearly: Number(config.baselineYearly) || 0
         };
 
-        // Apply baselines if new values entered
-        let baselineApplied = node.context().get("baselineApplied") || false;
-        if (!baselineApplied || baselineDaily || baselineWeekly || baselineMonthly || baselineYearly) {
-            baseline.daily   += baselineDaily;
-            baseline.weekly  += baselineWeekly;
-            baseline.monthly += baselineMonthly;
-            baseline.yearly  += baselineYearly;
-
-            storedBaselines = {
-                daily: baselineDaily,
-                weekly: baselineWeekly,
-                monthly: baselineMonthly,
-                yearly: baselineYearly
-            };
+        if (storedBaselines.daily || storedBaselines.weekly || storedBaselines.monthly || storedBaselines.yearly) {
+            baseline.daily   += storedBaselines.daily;
+            baseline.weekly  += storedBaselines.weekly;
+            baseline.monthly += storedBaselines.monthly;
+            baseline.yearly  += storedBaselines.yearly;
 
-            node.context().set("baselineApplied", true);
+            node.context().set("baseline", baseline);
             node.context().set("storedBaselines", storedBaselines);
-
-            // Cosmetic blanking: clear config fields so they show empty in UI
-            config.baselineDaily   = "";
-            config.baselineWeekly  = "";
-            config.baselineMonthly = "";
-            config.baselineYearly  = "";
         }
 
-        // Load last timestamp or initialize
+        // Last timestamp
         let lastCheck = node.context().get("lastCheck") || new Date();
 
-        // Currency symbol helper
+        // Helpers
         function currencySymbol(code) {
             switch (code) {
                 case "USD": return "$";
@@ -70,14 +49,11 @@ module.exports = function(RED) {
                 default: return code;
             }
         }
-
-        // Safe rounding helper
         function round2(val) {
             if (val === null || val === undefined || isNaN(val)) return 0;
             return Number(val.toFixed(2));
         }
 
-        // Archive yearly totals
         function archiveYearly(total, year) {
             try {
                 const archivePath = path.join(path.dirname(filePath), "yearly_archive.json");
@@ -85,11 +61,7 @@ module.exports = function(RED) {
                 if (fs.existsSync(archivePath)) {
                     archive = JSON.parse(fs.readFileSync(archivePath));
                 }
-                archive.push({
-                    year: year,
-                    total_kWh: total,
-                    timestamp: new Date().toISOString()
-                });
+                archive.push({ year, total_kWh: total, timestamp: new Date().toISOString() });
                 fs.writeFileSync(archivePath, JSON.stringify(archive, null, 2));
             } catch (err) {
                 node.error("Failed to archive yearly total: " + err);
@@ -99,26 +71,14 @@ module.exports = function(RED) {
         // Rollover logic
         function checkRollover() {
             let now = new Date();
+            let stored = node.context().get("storedBaselines") || { daily:0, weekly:0, monthly:0, yearly:0 };
 
-            // Daily rollover (midnight)
-            if (now.getDate() !== lastCheck.getDate()) {
-                baseline.daily = storedBaselines.daily;
-            }
-
-            // Weekly rollover (Sunday → Monday)
-            if (now.getDay() === 1 && lastCheck.getDay() !== 1) {
-                baseline.weekly = storedBaselines.weekly;
-            }
-
-            // Monthly rollover (1st of month)
-            if (now.getMonth() !== lastCheck.getMonth()) {
-                baseline.monthly = storedBaselines.monthly;
-            }
-
-            // Yearly rollover (Jan 1)
+            if (now.getDate() !== lastCheck.getDate()) baseline.daily = stored.daily;
+            if (now.getDay() === 1 && lastCheck.getDay() !== 1) baseline.weekly = stored.weekly;
+            if (now.getMonth() !== lastCheck.getMonth()) baseline.monthly = stored.monthly;
             if (now.getFullYear() !== lastCheck.getFullYear()) {
                 archiveYearly(baseline.yearly, lastCheck.getFullYear());
-                baseline.yearly = storedBaselines.yearly;
+                baseline.yearly = stored.yearly;
             }
 
             lastCheck = now;
@@ -127,24 +87,25 @@ module.exports = function(RED) {
         }
         setInterval(checkRollover, 60000);
 
+        // Input handler
         node.on('input', function(msg) {
             let now = new Date();
             let durationHours = (now - lastCheck) / (1000 * 3600);
-            lastCheck = now;
-            node.context().set("lastCheck", lastCheck);
-
-            let power = Number(msg.payload) || 0;
-            let power_kW = (inputUnit === "W") ? power / 1000 : power;
 
-            let energyIncrement = power_kW * durationHours;
+            let power = Number(msg.payload);
+            if (!isNaN(power) && durationHours > 0) {
+                let power_kW = (inputUnit === "W") ? power / 1000 : power;
+                let energyIncrement = power_kW * durationHours;
 
-            // Add increment to daily
-            baseline.daily += energyIncrement;
+                baseline.daily   += energyIncrement;
+                baseline.weekly  += energyIncrement;
+                baseline.monthly += energyIncrement;
+                baseline.yearly  += energyIncrement;
+            }
 
-            // Continuous absorption
-            baseline.weekly  += energyIncrement;
-            baseline.monthly += energyIncrement;
-            baseline.yearly  += energyIncrement;
+            lastCheck = now;
+            node.context().set("lastCheck", lastCheck);
+            node.context().set("baseline", baseline);
 
             msg.payload = {
                 daily_kWh: round2(baseline.daily),
@@ -158,18 +119,10 @@ module.exports = function(RED) {
                 currency: currencySymbol(currencyCode)
             };
 
-            // Ensure directory exists
             const dir = path.dirname(filePath);
-            if (!fs.existsSync(dir)) {
-                fs.mkdirSync(dir, { recursive: true });
-            }
-
-            // Write snapshot safely
-            try {
-                fs.writeFileSync(filePath, JSON.stringify(msg.payload, null, 2));
-            } catch (err) {
-                node.error("Failed to write to file: " + err);
-            }
+            if (!fs.existsSync(dir)) fs.mkdirSync(dir, { recursive: true });
+            try { fs.writeFileSync(filePath, JSON.stringify(msg.payload, null, 2)); }
+            catch (err) { node.error("Failed to write to file: " + err); }
 
             node.send(msg);
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-red-contrib-energymeterplus",
-  "version": "0.2.3",
+  "version": "0.2.5",
   "description": "A custom Node-RED node that integrates power readings into energy totals with persistence and cost calculation.",
   "author": "Arcfrankye",
   "license": "MIT",