signalk-rec-bms 1.1.0 → 1.1.2

package/README.md

@@ -1,6 +1,6 @@
 # signalk-rec-bms
 
-![alt text](https://github.com/ofernander/signalk-rec-bms/blob/main/public/assets/images/rec-sk.png)
+![alt text](https://raw.githubusercontent.com/ofernander/signalk-rec-bms/refs/heads/main/public/assets/images/rec-sk.png)
 
 ## Features
 
@@ -24,11 +24,11 @@ Tested with the 4 cell ABMS model only
   
 #### Serial connection:
 
-- RS485 to USB adapter with 5v output. The REC-BMS serial chip is not powered internally and you must supply 5v to power it. 
+- RS485 to USB adapter with 3v output. The REC-BMS serial chip is not powered internally and you must supply 3v to power it. 
 
-![alt text](https://github.com/ofernander/signalk-rec-bms/blob/main/public/assets/images/rs485.jpg)
+![alt text](https://raw.githubusercontent.com/ofernander/signalk-rec-bms/refs/heads/main/public/assets/images/rs485.jpg)
 
-![alt text](https://github.com/ofernander/signalk-rec-bms/blob/main/public/assets/images/bms_serial.png)
+![alt text](https://raw.githubusercontent.com/ofernander/signalk-rec-bms/refs/heads/main/public/assets/images/bms_serial.png)
 
 ## Dependencies
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "signalk-rec-bms",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "SignalK plugin for REC-BMS",
   "main": "plugin/index.js",
   "scripts": {

package/plugin/lib/schema.js

@@ -20,7 +20,7 @@ module.exports = {
         websocketURL: {
           title: "WebSocket URL",
           type: "string",
-          default: "ws://192.168.88.10/ws"
+          default: "ws://WiFi_module_IP/ws"
         }
       }
     },

package/plugin/lib/serial/modules/array.js

@@ -35,8 +35,8 @@ function parseLCD1Response(input) {
       current:     packet.readFloatLE(12),
       temperature: packet.readFloatLE(16),
       packVoltage: packet.readFloatLE(20),
-      soc:         packet.readFloatLE(24) * 100,
-      soh:         packet.readFloatLE(28) * 100
+      soc:         packet.readFloatLE(24),
+      soh:         packet.readFloatLE(28)
     }
   };
   return result;

package/plugin/lib/serial/modules/derived.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = function Derived(app, sendDeltaCallback, config) {
   app.debug("[DERIVED] Module instantiated with config: " + JSON.stringify(config));
   let voltage, current, ampHourUsed, capacity;
@@ -11,13 +13,29 @@ module.exports = function Derived(app, sendDeltaCallback, config) {
     if ([voltage, current, ampHourUsed, capacity].every(v => typeof v === 'number')) {
       const power = voltage * current;
       const ahRemaining = capacity - ampHourUsed;
+
+      const values = [
+        { path: prefix + 'power',             value: power },
+        { path: prefix + 'ampHourRemaining', value: ahRemaining }
+      ];
+
+      if (current > 0 && ahRemaining < capacity) {
+        const missingAh = capacity - ahRemaining;
+        const timeToFull = (missingAh / current) * 3600;
+        values.push({ path: prefix + 'timeToFull', value: timeToFull });
+        values.push({ path: prefix + 'timeToEmpty', value: 0 });
+      } else if (current < 0 && ahRemaining > 0) {
+        const timeToEmpty = (ahRemaining / Math.abs(current)) * 3600;
+        values.push({ path: prefix + 'timeToFull', value: 0 });
+        values.push({ path: prefix + 'timeToEmpty', value: timeToEmpty });
+      } else {
+        // fallback when current is 0 or data is abnormal
+        values.push({ path: prefix + 'timeToFull', value: 0 });
+        values.push({ path: prefix + 'timeToEmpty', value: 0 });
+      }
+
       sendDeltaCallback({
-        updates: [{
-          values: [
-            { path: prefix + 'power',             value: power },
-            { path: prefix + 'ampHourRemaining', value: ahRemaining }
-          ]
-        }]
+        updates: [{ values }]
       });
     }
   }
@@ -56,7 +74,7 @@ module.exports = function Derived(app, sendDeltaCallback, config) {
   });
 
   function trySendCellDiff() {
-    const keys = Object.keys(cellVoltages).map(k => parseInt(k,10));
+    const keys = Object.keys(cellVoltages).map(k => parseInt(k, 10));
     if (keys.length === totalCells) {
       const vals = keys.map(i => cellVoltages[i]);
       const minV = Math.min(...vals);
@@ -68,6 +86,8 @@ module.exports = function Derived(app, sendDeltaCallback, config) {
           ]
         }]
       });
+    } else {
+      setTimeout(trySendCellDiff, 10);
     }
   }
 

package/plugin/lib/websocket/derived.js

@@ -0,0 +1,78 @@
+"use strict";
+
+// Cache latest values across messages
+let latestMaster = null;
+let latestSettings = null;
+
+module.exports = function(parsedData, prefix) {
+  const values = [];
+
+  // Update cache if master data is present
+  if (parsedData?.bms_array?.master) {
+    latestMaster = parsedData.bms_array.master;
+  }
+
+  // Update cache if settings data is present
+  if (parsedData?.type === "settings") {
+    latestSettings = parsedData;
+  }
+
+  // If we don't have both yet, don't compute anything
+  if (!latestMaster || !latestSettings) return values;
+
+  const master = latestMaster;
+  const settings = latestSettings;
+
+  // power = voltage * current
+  if (typeof master.vbat === "number" && typeof master.ibat === "number") {
+    values.push({
+      path: `${prefix}.power`,
+      value: master.vbat * master.ibat
+    });
+  }
+
+  // cellVoltageDifference = max - min
+  if (typeof master.maxcell === "number" && typeof master.mincell === "number") {
+    values.push({
+      path: `${prefix}.cellVoltageDifference`,
+      value: master.maxcell - master.mincell
+    });
+  }
+
+  // ampHourRemaining = capacity - ampHourUsed
+  if (typeof settings.capa === "number" && typeof settings.Ah === "number") {
+    values.push({
+      path: `${prefix}.ampHourRemaining`,
+      value: settings.capa - settings.Ah
+    });
+  }
+
+  // timeToFull / timeToEmpty in seconds
+  if (
+    typeof settings.capa === "number" &&
+    typeof settings.Ah === "number" &&
+    typeof master.ibat === "number"
+  ) {
+    const capacity = settings.capa;
+    const current = master.ibat;
+    const soc = typeof master.soc === "number" ? master.soc : null;
+    const remainingAh = (soc !== null) ? (soc / 100) * capacity : (capacity - settings.Ah);
+
+    if (current > 0 && remainingAh < capacity) {
+      const missingAh = capacity - remainingAh;
+      const timeToFull = (missingAh / current) * 3600;
+      values.push({ path: `${prefix}.timeToFull`, value: timeToFull });
+      values.push({ path: `${prefix}.timeToEmpty`, value: 0 });
+    } else if (current < 0 && remainingAh > 0) {
+      const timeToEmpty = (remainingAh / Math.abs(current)) * 3600;
+      values.push({ path: `${prefix}.timeToFull`, value: 0 });
+      values.push({ path: `${prefix}.timeToEmpty`, value: timeToEmpty });
+    } else {
+      // idle or insufficient current
+      values.push({ path: `${prefix}.timeToFull`, value: 0 });
+      values.push({ path: `${prefix}.timeToEmpty`, value: 0 });
+    }
+  }
+
+  return values;
+};

package/plugin/lib/websocket/settings.js

@@ -8,11 +8,6 @@ module.exports = function(parsedData, prefix, config) {
   const settings = parsedData;
   const values = []; 
 
-  // Derived data - NOT FROM BMS
-  const AhR = (parsedData.capa !== undefined && parsedData.Ah !== undefined)
-    ? parsedData.capa - parsedData.Ah
-    : null;
-  values.push({ path: `${prefix}.ampHourRemaining`, value: AhR });
   // Data from WiFi module WebSocket under SETTINGS message 
   values.push({ path: `${prefix}.address`, value: parsedData.addr });
   values.push({ path: `${prefix}.ampHourUsed`, value: parsedData.Ah });

package/plugin/lib/websocket/status.js

@@ -23,15 +23,10 @@ module.exports = function(parsedData, prefix, config) {
     return null;
   }
 
-  // Derived data - NOT FROM BMS
-  const power = (master.vbat !== undefined && master.ibat !== undefined)
-    ? master.vbat * master.ibat
-    : null;
-  values.push({ path: `${prefix}.power`, value: power });
   // Data from WiFi module WebSocket under SETTINGS message 
   values.push({ path: `${prefix}.date`, value: master.date });
   values.push({ path: `${prefix}.time`, value: master.time });
-  values.push({ path: `${prefix}.cellVoltageDifference`, value: master.maxcell - master.mincell });
+  //values.push({ path: `${prefix}.cellVoltageDifference`, value: master.maxcell - master.mincell });
   values.push({ path: `${prefix}.current`, value: master.ibat });
   values.push({ path: `${prefix}.errorAddress`, value: master.erro.addr });
   values.push({ path: `${prefix}.errorConnectionState`, value: master.erro.con_st });
@@ -42,7 +37,7 @@ module.exports = function(parsedData, prefix, config) {
   values.push({ path: `${prefix}.minCellVoltage`, value: master.mincell });
   values.push({ path: `${prefix}.stateOfCharge`, value: master.soc });
   values.push({ path: `${prefix}.stateOfHealth`, value: master.soh });
-  values.push({ path: `${prefix}.timeRemaining`, value: extractMinutes(master.time_remaining) });
+  //values.push({ path: `${prefix}.timeRemaining`, value: extractMinutes(master.time_remaining) });
   values.push({ path: `${prefix}.voltage`, value: master.vbat });
   values.push({ path: `${prefix}.bmsTemperature`, value: slave.temp_bms });
   values.push({ path: `${prefix}.slaveAddress`, value: slave.address });

package/plugin/lib/websocket/ws.js

@@ -2,6 +2,7 @@
 const WebSocket = require('ws');
 const status = require('./status');
 const settings = require('./settings');
+const derived = require('./derived');
 const schema = require('../schema');
 
 module.exports = function(app, publishDelta) {
@@ -84,6 +85,11 @@ module.exports = function(app, publishDelta) {
           }
         }
 
+        // Add derived values
+        const derivedValues = derived(parsedData, prefix) || [];
+        delta.updates[0].values.push(...derivedValues);
+
+        // Filter out null/undefined
         delta.updates[0].values = delta.updates[0].values.filter(item => item.value !== null && item.value !== undefined);
 
         if (delta.updates[0].values.length > 0) {
@@ -111,9 +117,8 @@ module.exports = function(app, publishDelta) {
     });
 
     ws.on('error', (error) => {
-      app.error(`[WEBSOCKET] WebSocket error: ${error.message}`);
       app.setPluginStatus(`Error: ${error.message} - ERROR`);
-      app.debug("[WEBSOCKET] WebSocket error occurred, terminating connection.");
+      app.debug(`[WEBSOCKET] WebSocket error: "${error.message}", terminating connection.`);
       clearInterval(pingInterval);
       if (ws) ws.terminate();
       if (shouldReconnect) {

package/public/index.html

@@ -36,7 +36,7 @@
       </div>
       <div class="metric-box">
         <div class="metric-label">State of Charge</div>
-        <div class="metric-value" id="soc">-- %</div>
+        <div class="metric-value" id="stateOfCharge">-- %</div>
       </div>
       <div class="metric-box">
         <div class="metric-label">Pack Voltage</div>

package/public/js/main.js

@@ -14,7 +14,8 @@ class BMSSystem {
       voltage: null,
       current: null,
       power: null,
-      soc: null,
+      stateOfCharge: null,
+      stateOfHealth: null,
       cellVoltages: {},
       cellResistances: {},
       cellVoltageDifference: null,
@@ -38,11 +39,12 @@ class BMSSystem {
 
   cacheDomElements() {
     const metrics = [
-      'voltage', 'current', 'power', 'soc', 'cellVoltageDifference',
-      'bmsTemperature', 'cellTemperature1', 'maxCellVoltage', 'minCellVoltage',
-      'ampHourRemaining', 'ampHourUsed', 'capacity'
+      'voltage', 'current', 'power', 'stateOfCharge', 'stateOfHealth',
+      'cellVoltageDifference', 'bmsTemperature', 'cellTemperature1',
+      'maxCellVoltage', 'minCellVoltage', 'ampHourRemaining',
+      'ampHourUsed', 'capacity'
     ];
-    
+
     metrics.forEach(metric => {
       this.dom.metricElements[metric] = document.getElementById(metric);
       if (!this.dom.metricElements[metric]) {
@@ -54,7 +56,7 @@ class BMSSystem {
   setupTabSwitching() {
     const statusTab = document.getElementById('nav-status');
     const settingsTab = document.getElementById('nav-settings');
-    
+
     if (!statusTab || !settingsTab) {
       console.warn("Tab elements not found");
       return;
@@ -68,7 +70,7 @@ class BMSSystem {
   switchView(view) {
     const statusView = document.getElementById('status-view');
     const settingsView = document.getElementById('settings-view');
-    
+
     if (view === 'status') {
       statusView?.classList.add('active');
       settingsView?.classList.remove('active');
@@ -90,8 +92,23 @@ class BMSSystem {
     setValue(this.dom.metricElements.voltage, this.liveData.voltage, "V");
     setValue(this.dom.metricElements.current, this.liveData.current, "A");
     setValue(this.dom.metricElements.power, this.liveData.power, "W");
-    setValue(this.dom.metricElements.soc, this.liveData.soc, "%");
-    setValue(this.dom.metricElements.cellVoltageDifference, this.liveData.cellVoltageDifference, "V");
+
+    setValue(
+      this.dom.metricElements.stateOfCharge,
+      this.liveData.stateOfCharge !== null ? this.liveData.stateOfCharge * 100 : null,
+      "%"
+    );
+    setValue(
+      this.dom.metricElements.stateOfHealth,
+      this.liveData.stateOfHealth !== null ? this.liveData.stateOfHealth * 100 : null,
+      "%"
+    );
+
+    if (this.dom.metricElements.cellVoltageDifference) {
+      const v = this.liveData.cellVoltageDifference;
+      this.dom.metricElements.cellVoltageDifference.textContent = v !== null ? `${v.toFixed(4)}V` : "--";
+    }
+
     setValue(this.dom.metricElements.bmsTemperature, this.liveData.bmsTemperature, "°C");
     setValue(this.dom.metricElements.cellTemperature1, this.liveData.cellTemperature1, "°C");
     setValue(this.dom.metricElements.maxCellVoltage, this.liveData.maxCellVoltage, "V");