nodejs-poolcontroller 7.5.1 → 7.7.0

package/controller/comms/messages/status/ChlorinatorStateMessage.ts

@@ -21,13 +21,19 @@ import { sys, ControllerType, Chlorinator } from "../../../Equipment";
 export class ChlorinatorStateMessage {
     public static process(msg: Inbound) {
         if (msg.protocol === Protocol.Chlorinator) {
-            let chlor: Chlorinator;
-            let cstate: ChlorinatorState;
-            
+            // RKS: 03-29-22 A lot of water has gone under the bridge at this point and we know much more.  First there are two types of messages.  Those inbound
+            // messages that are being sent to the chlorinator as commands and responses from the chlorinator.  We have also determined that the chlorinator does
+            // not actually support a RS485 address.  So njsPC can declare as many ports as required to communicate with multiple chlorinators.
+            //
+            // So instead of matching the chlorinator up with a specific id we need to match on the port as well for the inbound message.  OCPs always operate on
+            // portId = 0.  If the portId is > 0 then it is considered an Aux port.  At this point you can only control 1 chlorinator per port so in order to
+            // control more than one chlorinator there needs to be only one chlorinator on each port.            
+            let chlor: Chlorinator = sys.chlorinators.findItemByPortId(msg.portId || 0);
+            if (typeof chlor === 'undefined' || chlor.isActive === false) return; // Bail out of here if we don't find an active chlorinator.
+            let cstate: ChlorinatorState = state.chlorinators.getItemById(chlor.id, true);
             if (msg.dest >= 1 && msg.dest <= 4) {
-                // RKS: The dest for these message are 80+ in raw terms.  The msg object translates these into 1-4 for the installed chlorinators.  This message
-                // is from the OCP to the chlorinator.
-                cstate = state.chlorinators.getItemById(msg.dest, true);
+                // RKS: This message is from the OCP to the chlorinator.  NOTE: We will not see these messages when the communication is coming from njsPC on any
+                // comms port.  The processing for no comms is done in the Nixe control when the message is sent.
                 if (typeof cstate.lastComm === 'undefined') cstate.lastComm = new Date(1970, 0, 1, 0, 0, 0, 0).getTime();
                 // RG: I was getting some time deltas of 25-30s and bumped this up
                 else if (cstate.lastComm + (30 * 1000) < new Date().getTime()) {
@@ -35,27 +41,24 @@ export class ChlorinatorStateMessage {
                     cstate.status = 128;
                 }
                 chlor = sys.chlorinators.getItemById(msg.dest, true);
-                chlor.address = msg.dest + 79;
+                chlor.address = msg.dest + 79; // Theoretically, this will always be 80.
                 if (typeof chlor.isActive === 'undefined') cstate.isActive = chlor.isActive = true;
             }
             else {
-                // Message from chlorinator
-                cstate = state.chlorinators.getItemById(msg.dest + 1, true);
-                chlor = sys.chlorinators.getItemById(msg.dest + 1, true);
+                // Response message from chlorinator.  If the chlorinator is speaking to us then we need to hear it and clear the status when
+                // the previous status was no comms.
                 cstate.lastComm = new Date().getTime();
                 if (cstate.status === 128) cstate.status = 0;
             }
             cstate.body = chlor.body;
             switch (msg.action) {
-                case 0: // request status (0): [16,2,80,0][0][98,16,3]
+                case 0: // Set control OCP->Chlorinator: [16,2,80,0][0][98,16,3]
                     break;
-                case 1: // response to request status: [16,2,0,1][0,0][19,16,3]
-                    {
-                        // let chlor = sys.chlorinators.getItemById(1, true);
-                        // chlor.isActive = true;
-                        break;
-                    }
-                case 3: {
+                case 1: // Ack control Chlorinator->OCP response: [16,2,0,1][0,0][19,16,3]
+                    // let chlor = sys.chlorinators.getItemById(1, true);
+                    // chlor.isActive = true;
+                    break;
+                case 3: // Chlorinator->OCP response for Get Model: [16,2,0,3][0,73,110,116,101,108,108,105,99,104,108,111,114,45,45,52,48][188,16,3]
                     // RKS: 07-16-20 -- It appears that this message doesn't always come.  It is very likely that the newer versions of IntelliChlor
                     // do not respond to the 20 message.  There have been multiple instances of this with recent versions of IntelliChlor. As a result,
                     // don't overwrite the name should the user set it.
@@ -65,12 +68,11 @@ export class ChlorinatorStateMessage {
                     // This is the model number of the chlorinator and the address is actually the second byte.
                     let name = msg.extractPayloadString(1, 16);
                     if (typeof chlor.name === 'undefined' || chlor.name === '') chlor.name = cstate.name = name;
-                    if (typeof chlor.model === 'undefined') chlor.model = sys.board.valueMaps.chlorinatorModel.getValue(name.toLowerCase()); 
+                    if (typeof chlor.model === 'undefined') chlor.model = sys.board.valueMaps.chlorinatorModel.getValue(name.toLowerCase());
                     cstate.isActive = chlor.isActive;
                     state.emitEquipmentChanges();
                     break;
-                }
-                case 17: {
+                case 17: // OCP->Chlorinator set output. [16,2,80,17][15][130,16,3]
                     // If the chlorinator is no longer talking to us then clear the current output.
                     if (cstate.status === 128) cstate.currentOutput = 0;
                     cstate.targetOutput = msg.extractPayloadByte(0);
@@ -78,13 +80,16 @@ export class ChlorinatorStateMessage {
                         // Some dumbass is trying to change our output.  We need to set it back to 0.
                         sys.board.chlorinator.setChlorAsync({ id: chlor.id, disabled: chlor.disabled, superChlor: false, superChlorHours: 0 });
                     }
-                    else if (chlor.isDosing && cstate.targetOutput !== 100){
+                    else if (chlor.isDosing && cstate.targetOutput !== 100) {
                         sys.board.chlorinator.setChlorAsync({ id: chlor.id, isDosing: chlor.isDosing });
                     }
                     state.emitEquipmentChanges();
                     break;
-                }
-                case 21: {
+                case 21: // Set output OCP->Chlorinator [16,2,80,21][0][119,16,3]
+                    // RKS: 03-29-22 There is something absolutley wrong about the understanding of this message.  It has to do with the fact that you could never set
+                    // the output percentage to anything greater than 25.6% if this were the case.  I assume this has something to do with the
+                    // way IntelliChlor works.  Values between 0 and 20% can be fractional.  In this case the OCP would send a 21 instead of a 17 with
+                    // the fractional value.  Anything above that value will be sent in a 17 message.
                     // Set Cell Output / 10
                     // This packet is coming through differently on the IntelliConnect.
                     // eg 13:42:31.304 VERBOSE Msg# 1531   Controller --> Salt cell: Set current output to 1.6 %: 16,2,80,21,0,119,16,3
@@ -95,34 +100,31 @@ export class ChlorinatorStateMessage {
                         // Some dumbass is trying to change our output.  We need to set it back to 0.
                         sys.board.chlorinator.setChlorAsync({ id: chlor.id, disabled: chlor.disabled, superChlor: false, superChlorHours: 0 });
                     }
-                    else if (chlor.isDosing && cstate.targetOutput !== 100){
+                    else if (chlor.isDosing && cstate.targetOutput !== 100) {
                         sys.board.chlorinator.setChlorAsync({ id: chlor.id, isDosing: chlor.isDosing });
                     }
                     state.emitEquipmentChanges();
                     break;
-                }
-                case 18: {
+                case 18:
                     // Response to Set Salt Output (17 & 20)
                     // The most common failure with IntelliChlor is that the salt level stops reporting.  Below should allow it to be fed from an alternate
                     // source like REM.
-                    if(!chlor.ignoreSaltReading) cstate.saltLevel = msg.extractPayloadByte(0) * 50 || cstate.saltLevel || 0;
+                    if (!chlor.ignoreSaltReading) cstate.saltLevel = msg.extractPayloadByte(0) * 50 || cstate.saltLevel || 0;
                     cstate.status = (msg.extractPayloadByte(1) & 0x007F); // Strip off the high bit.  The chlorinator does not actually report this. 
                     cstate.currentOutput = chlor.disabled ? 0 : cstate.setPointForCurrentBody;
                     state.emitEquipmentChanges();
                     break;
-                }
-                case 19: {
+                case 19:
                     // This is an iChlor message with no payload.  Perhaps simply a keep alive for the iChlor.
                     // [16, 2, 80, 19][117, 16, 3]
                     break;
-                }
-                case 20: {
-                    // Get version
-                    chlor.type = cstate.type = msg.extractPayloadByte(0);
+                case 20:
+                    // OCP->Chlorinator Get model [16,2,80,20][0][118,16,3]
+                    if (typeof chlor.type === 'undefined') chlor.type = msg.extractPayloadByte(0);
+                    cstate.type = chlor.type;
                     state.emitEquipmentChanges();
                     break;
-                }
-                case 22: {
+                case 22: // Chlorinator->OCP this is actually an iChlor message and has no bearing on IntelliConnect.
                     // temp and output as seen from IntelliConnect.  
                     // Issue #157 - https://github.com/tagyoureit/nodejs-poolController/issues/157
                     // [10 02, 10, 16], [00, 0f, 49, 00,05, 10], [85,  10,03] = hex
@@ -141,57 +143,7 @@ export class ChlorinatorStateMessage {
                     }
                     state.emitEquipmentChanges();
                     break;
-                }
             }
         }
-        // question: does IntelliCenter ever broadcast Chlorinator packet?  Answer: Never.  My guess is that this is actually 
-        // a configuration message rather than a status message.  Also, IntelliCenter has a 204 extension status that contains
-        // the current countdown timer for the superChlor in the last 2 bytes.  Perhaps this is the equivalent.
-        //else if (msg.protocol === Protocol.Broadcast) {
-        //    if (!state.isInitialized) return;
-        //    // sample packet
-        //    // [165,33,15,16,25,22],[1,10,128,29,132,0,73,110,116,101,108,108,105,99,104,108,111,114,45,45,52,48],[7,231]
-        //    let chlorId = 1;
-        //    let chlor = sys.chlorinators.getItemById(chlorId, true);
-        //    if (chlor.isVirtual) { return; } // shouldn't get here except for testing Chlor on *Touch system.
-        //    // installed = (aaaaaaa)1 so 1 = installed
-        //    chlor.isActive = (msg.extractPayloadByte(0) & 0x01) === 1;
-        //    if (chlor.isActive) {
-        //        // RSG : making the assumption here that the chlorinator will be tied to the pool in any system that is not a shared body
-        //        sys.equipment.maxBodies >= 1 || sys.equipment.shared === true ? chlor.body = 32 : chlor.body = 0;
-        //        // outputSpaPercent field is aaaaaaab (binary) where aaaaaaa = % and b===installed (0=no,1=yes)
-        //        // eg. a value of 41 is 00101001
-        //        // spa percent = 0010100(b) so 10100 = 20
-        //        if (!chlor.disabled) {
-        //            // RKS: We don't want these setpoints if our chem controller disabled the
-        //            // chlorinator.  These should be 0 anyway.
-        //            chlor.spaSetpoint = msg.extractPayloadByte(0) >> 1;
-        //            chlor.poolSetpoint = msg.extractPayloadByte(1);
-        //        }
-        //        chlor.address = chlor.id + 79;
-        //        chlor.superChlor = msg.extractPayloadByte(5) > 0;
-        //        chlor.superChlorHours = msg.extractPayloadByte(5);
-        //        chlor.name = msg.extractPayloadString(6, 16);
-        //        let schlor = state.chlorinators.getItemById(chlorId, true);
-        //        // The most common failure with IntelliChlor is that the salt level stops reporting.  Below should allow it to be fed from an alternate
-        //        // source like REM.
-        //        schlor.saltLevel = msg.extractPayloadByte(3) * 50 || schlor.saltLevel || 0;
-        //        schlor.status = msg.extractPayloadByte(4) & 0x007F; // Strip off the high bit.  The chlorinator does not actually report this.;
-        //        schlor.lastComm = new Date().getTime();  // rely solely on "true" chlor messages for this?
-        //        schlor.poolSetpoint = chlor.poolSetpoint;
-        //        schlor.spaSetpoint = chlor.spaSetpoint;
-        //        schlor.superChlor = chlor.superChlor;
-        //        schlor.superChlorHours = chlor.superChlorHours;
-        //        schlor.name = chlor.name;
-        //        schlor.body = chlor.body;
-        //        if (state.temps.bodies.getItemById(1).isOn) schlor.targetOutput = chlor.disabled ? 0 : chlor.poolSetpoint;
-        //        else if (state.temps.bodies.getItemById(2).isOn) schlor.targetOutput = chlor.disabled ? 0 : chlor.spaSetpoint;
-        //        state.emitEquipmentChanges();
-        //    }
-        //    else {
-        //        sys.chlorinators.removeItemById(chlorId);
-        //        state.chlorinators.removeItemById(chlorId);
-        //    }
-        //}
     }
 }

package/controller/comms/messages/status/EquipmentStateMessage.ts

@@ -357,17 +357,41 @@ export class EquipmentStateMessage {
                             tbody.setPoint = cbody.setPoint;
                             tbody.name = cbody.name;
                             tbody.circuit = cbody.circuit = 6;
-                            tbody.heatMode = cbody.heatMode = msg.extractPayloadByte(22) & 0x03;
+
+                            //RKS: This heat mode did not include all the bits necessary for hybrid heaters
+                            //tbody.heatMode = cbody.heatMode = msg.extractPayloadByte(22) & 0x03;
+                            tbody.heatMode = cbody.heatMode = msg.extractPayloadByte(22) & 0x33;
                             let heatStatus = sys.board.valueMaps.heatStatus.getValue('off');
                             if (tbody.isOn) {
-                                //const heaterActive = (msg.extractPayloadByte(10) & 0x0C) === 12;
-                                //const solarActive = (msg.extractPayloadByte(10) & 0x30) === 48;
-                                const heaterActive = (msg.extractPayloadByte(10) & 0x04) === 0x04;
-                                const solarActive = (msg.extractPayloadByte(10) & 0x10) === 0x10;
-                                const cooling = solarActive && tbody.temp > tbody.setPoint;
-                                if (heaterActive) heatStatus = sys.board.valueMaps.heatStatus.getValue('heater');
-                                if (cooling) heatStatus = sys.board.valueMaps.heatStatus.getValue('cooling');
-                                else if (solarActive) heatStatus = sys.board.valueMaps.heatStatus.getValue('solar');
+                                if (tbody.heaterOptions.hybrid > 0) {
+                                    // ETi When heating with
+                                    // Heatpump (1) = 12    H:true S:false C:false
+                                    // Gas (2) = 48         H:false S:true C:false
+                                    // Hybrid (3) = 48      H:true S:false C:false
+                                    // Dual (16) = 60       H:true S:true C:false
+                                    // What this means is that Touch actually treats the heat status as either heating with
+                                    // the primary heater for the body or the secondary.  In the case of a hybrid heater
+                                    // the primary is a heatpump and the secondary is gas.  In the case of gas + solar or gas + heatpump
+                                    // the gas heater is the primary and solar or heatpump is the secondary.   So we need to dance a little bit
+                                    // here.  We do this by checking the heater options.
+
+                                    // This can be the only heater solar cannot be installed with this.
+                                    let byte = msg.extractPayloadByte(10);
+                                    // Either the primary, secondary, or both is engaged.
+                                    if ((byte & 0x14) === 0x14) heatStatus = sys.board.valueMaps.heatStatus.getValue('dual');
+                                    else if (byte & 0x10) heatStatus = sys.board.valueMaps.heatStatus.getValue('heater');
+                                    else if (byte & 0x04) heatStatus = sys.board.valueMaps.heatStatus.getValue('hpheat');
+                                }
+                                else {
+                                    //const heaterActive = (msg.extractPayloadByte(10) & 0x0C) === 12;
+                                    //const solarActive = (msg.extractPayloadByte(10) & 0x30) === 48;
+                                    const heaterActive = (msg.extractPayloadByte(10) & 0x04) === 0x04;
+                                    const solarActive = (msg.extractPayloadByte(10) & 0x10) === 0x10;
+                                    const cooling = solarActive && tbody.temp > tbody.setPoint;
+                                    if (heaterActive) heatStatus = sys.board.valueMaps.heatStatus.getValue('heater');
+                                    if (cooling) heatStatus = sys.board.valueMaps.heatStatus.getValue('cooling');
+                                    else if (solarActive) heatStatus = sys.board.valueMaps.heatStatus.getValue('solar');
+                                }
                             }
                             tbody.heatStatus = heatStatus;
                             sys.board.schedules.syncScheduleHeatSourceAndSetpoint(cbody, tbody);
@@ -380,21 +404,32 @@ export class EquipmentStateMessage {
                                 tbody.temp = sys.equipment.shared ? state.temps.waterSensor1 : state.temps.waterSensor2;
                                 tbody.isOn = true;
                             } else tbody.isOn = false;
-                            tbody.heatMode = cbody.heatMode = (msg.extractPayloadByte(22) & 0x0C) >> 2;
+                            //RKS: This heat mode did not include all the bits necessary for hybrid heaters
+                            //tbody.heatMode = cbody.heatMode = (msg.extractPayloadByte(22) & 0x0C) >> 2;
+                            tbody.heatMode = cbody.heatMode = (msg.extractPayloadByte(22) & 0xCC) >> 2;
                             tbody.setPoint = cbody.setPoint;
                             tbody.name = cbody.name;
                             tbody.circuit = cbody.circuit = 1;
                             let heatStatus = sys.board.valueMaps.heatStatus.getValue('off');
                             if (tbody.isOn) {
-                                //const heaterActive = (msg.extractPayloadByte(10) & 0x0C) === 12;
-                                //const solarActive = (msg.extractPayloadByte(10) & 0x30) === 48;
-                                const heaterActive = (msg.extractPayloadByte(10) & 0x08) === 0x08;
-                                const solarActive = (msg.extractPayloadByte(10) & 0x20) === 0x20;
-
-                                const cooling = solarActive && tbody.temp > tbody.setPoint;
-                                if (heaterActive) heatStatus = sys.board.valueMaps.heatStatus.getValue('heater');
-                                if (cooling) heatStatus = sys.board.valueMaps.heatStatus.getValue('cooling');
-                                else if (solarActive) heatStatus = sys.board.valueMaps.heatStatus.getValue('solar');
+                                if (tbody.heaterOptions.hybrid > 0) {
+                                    // This can be the only heater solar cannot be installed with this.
+                                    let byte = msg.extractPayloadByte(10);
+                                    // Either the primary, secondary, or both is engaged.
+                                    if ((byte & 0x28) === 0x28) heatStatus = sys.board.valueMaps.heatStatus.getValue('dual');
+                                    else if (byte & 0x20) heatStatus = sys.board.valueMaps.heatStatus.getValue('heater');
+                                    else if (byte & 0x08) heatStatus = sys.board.valueMaps.heatStatus.getValue('hpheat');
+                                }
+                                else {
+                                    //const heaterActive = (msg.extractPayloadByte(10) & 0x0C) === 12;
+                                    //const solarActive = (msg.extractPayloadByte(10) & 0x30) === 48;
+                                    const heaterActive = (msg.extractPayloadByte(10) & 0x08) === 0x08;
+                                    const solarActive = (msg.extractPayloadByte(10) & 0x20) === 0x20;
+                                    const cooling = solarActive && tbody.temp > tbody.setPoint;
+                                    if (heaterActive) heatStatus = sys.board.valueMaps.heatStatus.getValue('heater');
+                                    if (cooling) heatStatus = sys.board.valueMaps.heatStatus.getValue('cooling');
+                                    else if (solarActive) heatStatus = sys.board.valueMaps.heatStatus.getValue('solar');
+                                }
                             }
                             tbody.heatStatus = heatStatus;
                             sys.board.schedules.syncScheduleHeatSourceAndSetpoint(cbody, tbody);
@@ -482,7 +517,8 @@ export class EquipmentStateMessage {
                 // pool
                 let tbody: BodyTempState = state.temps.bodies.getItemById(1, true);
                 let cbody: Body = sys.bodies.getItemById(1);
-                tbody.heatMode = cbody.heatMode = msg.extractPayloadByte(5) & 3;
+                // RKS: 02-26-22 - See communications doc for explanation of bits.  This needs to support UltraTemp ETi heatpumps.
+                tbody.heatMode = cbody.heatMode = msg.extractPayloadByte(5) & 0x33;
                 tbody.setPoint = cbody.setPoint = msg.extractPayloadByte(3);
                 tbody.coolSetpoint = cbody.coolSetpoint = msg.extractPayloadByte(9);
                 if (tbody.isOn) tbody.temp = state.temps.waterSensor1;
@@ -490,8 +526,8 @@ export class EquipmentStateMessage {
                 if (cbody.isActive) {
                     // spa
                     tbody = state.temps.bodies.getItemById(2, true);
-                    tbody.heatMode = cbody.heatMode =
-                        (msg.extractPayloadByte(5) & 12) >> 2;
+                    tbody.heatMode = cbody.heatMode = (msg.extractPayloadByte(5) & 0xCC) >> 2;
+                    //tbody.heatMode = cbody.heatMode = (msg.extractPayloadByte(5) & 12) >> 2;
                     tbody.setPoint = cbody.setPoint = msg.extractPayloadByte(4);
                     if (tbody.isOn) tbody.temp = state.temps.waterSensor2 = msg.extractPayloadByte(1);
                 }
@@ -689,7 +725,7 @@ export class EquipmentStateMessage {
                         case 144: // swim
                             sys.board.circuits.sequenceLightGroupAsync(grp.id, 'swim');
                             break;
-                        case 160: // swim
+                        case 160: // set
                             sys.board.circuits.sequenceLightGroupAsync(grp.id, 'set');
                             break;
                         case 190: // save

package/controller/comms/messages/status/HeaterStateMessage.ts

@@ -22,15 +22,50 @@ export class HeaterStateMessage {
     public static process(msg: Inbound) {
         if (msg.protocol === Protocol.Heater) {
             switch (msg.action) {
-                case 114: // This is a message from a master controlling the heater
+                case 112: // This is a message from a master controlling MasterTemp or UltraTemp ETi
+                    break;
+                case 114: // This is a message from a master controlling UltraTemp
+                    msg.isProcessed = true;
+                    break;
+                case 113:
+                    HeaterStateMessage.processHybridStatus(msg);
+                    break;
+                case 116:
+                    HeaterStateMessage.processMasterTempStatus(msg);
                     break;
                 case 115:
-                    HeaterStateMessage.processHeaterStatus(msg);
+                    HeaterStateMessage.processUltraTempStatus(msg);
+                    break;
+            }
+        }
+    }
+    public static processHeaterCommand(msg: Inbound) {
+        let heater: Heater = sys.heaters.getItemByAddress(msg.source);
+        // At this point there is no other configuration data for ET
+        if (sys.controllerType === ControllerType.EasyTouch) {
+            let htype = sys.board.valueMaps.heaterTypes.transform(heater.type);
+            switch (htype.name) {
+                case 'hybrid':
+                    heater.economyTime = msg.extractPayloadByte(3);
+                    heater.maxBoostTemp = msg.extractPayloadByte(4);
                     break;
             }
         }
     }
-    public static processHeaterStatus(msg: Inbound) {
+    public static processHybridStatus(msg: Inbound) {
+        //[165, 0, 16, 112, 113, 10][1, 1, 0, 0, 0, 0, 0, 0, 0, 0][1, 162]
+        let heater: Heater = sys.heaters.getItemByAddress(msg.source);
+        let sheater = state.heaters.getItemById(heater.id);
+        sheater.isOn = msg.extractPayloadByte(0) > 0;
+
+        let byte = msg.extractPayloadByte(2);
+        //sheater.isOn = byte >= 1;
+        //sheater.isCooling = byte === 2;
+        sheater.commStatus = 0;
+        state.equipment.messages.removeItemByCode(`heater:${heater.id}:comms`);
+        msg.isProcessed = true;
+    }
+    public static processUltraTempStatus(msg: Inbound) {
         // RKS: 07-03-21 - We only know byte 2 at this point for Ultratemp for the 115 message we are processing here.  The 
         // byte  description
         // ------------------------------------------------
@@ -60,4 +95,23 @@ export class HeaterStateMessage {
         state.equipment.messages.removeItemByCode(`heater:${heater.id}:comms`);
         msg.isProcessed = true;
     }
+    public static processMasterTempStatus(msg: Inbound) {
+        //[255, 0, 255][165, 0, 16, 112, 116, 23][67, 0, 0, 0, 0, 0, 0, 0, 68, 0, 0, 0, 10, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0][2, 66]
+        // Byte 1 is the indicator to which setpoint it is heating to.
+        // Byte 8 increments over time when the heater is on.
+        // Byte 13 looks like the mode the heater is in for instance it is in cooldown mode.
+        //  0 = Normal
+        //  2 = ??????
+        //  6 = Cooldown
+        // Byte 14 looks like the cooldown delay in minutes.
+        let heater: Heater = sys.heaters.getItemByAddress(msg.source);
+        let sheater = state.heaters.getItemById(heater.id);
+        let byte = msg.extractPayloadByte(1);
+        sheater.isOn = byte >= 1;
+        sheater.isCooling = false;
+        sheater.commStatus = 0;
+        state.equipment.messages.removeItemByCode(`heater:${heater.id}:comms`);
+        msg.isProcessed = true;
+    }
+
 }

package/controller/comms/messages/status/IntelliChemStateMessage.ts

@@ -87,6 +87,18 @@ export class IntelliChemStateMessage {
     }
     private static processState(msg: Inbound) {
         if (msg.source < 144 || msg.source > 158) return;
+
+        // Setup with CO2 dosing and IntelliChlor.  
+        //[2, 245, 2, 162, 2, 248, 2,  88, 0, 0, [0-9]
+        // 0, 26, 0, 0, 0, 0, 0, 0, 0, 0,        [10-19]
+        // 7, 0, 249, 1, 94, 0, 81, 0, 80, 57,   [20-29] 
+        // 0, 82, 0, 0, 162, 32, 80, 1, 0, 0, 0] [30-40]
+        // Setup with 2 Tanks
+        //[2, 238, 2, 208, 2, 248, 2, 188, 0, 0, 
+        // 0, 2, 0, 0, 0, 29, 0, 4, 0, 63,
+        // 2, 2, 157, 0, 25, 0, 0, 0, 90, 20, 
+        // 0, 83, 0, 0, 149, 0, 60, 1, 1, 0, 0]
+
         // This is an action 18 that comes from IntelliChem.  There is also a 147 that comes from an OCP but this is the raw data.
         //[165, 0, 16, 144, 18, 41][2,228,3,2,2,228,2,188,0,0,0,16,0,0,0,0,0,35,0,0,6,6,3,0,250,0,44,0,160,20,0,81,8,0,149,0,80,1,0,0,0]
         //      Bytes - Descrption
@@ -114,16 +126,31 @@ export class IntelliChemStateMessage {
         //      31 : Temperature
         //      32 : Alarms
         //      33 : Warnings pH Lockout, Daily Limit Reached, Invalid Setup, Chlorinator Comm error
-        //      34 : Dosing Status (pH Monitoring, ORP Mixing)
+        //      34 : Dosing Status/Doser Type (pH Monitoring, ORP Mixing)
         //      35 : Delays
         //      36-37 : Firmware = 80,1 = 1.080
         //      38 : Water Chemistry Warning (Corrosion...)
         //      39 : Unknown
         //      40 : Unknown
         let address = msg.source;
+
         // The address is king here.  The id is not.
         let chem = sys.chemControllers.getItemByAddress(address, true);
         let schem = state.chemControllers.getItemById(chem.id, true);
+
+        // Get the doser types and set up our capabilities
+        chem.ph.doserType = (msg.extractPayloadByte(34) & 0x03);
+        chem.orp.doserType = (msg.extractPayloadByte(34) & 0x0C) >> 2;
+        schem.ph.enabled = chem.ph.enabled = chem.ph.doserType !== 0;
+        schem.ph.enabled = chem.orp.enabled = chem.orp.doserType !== 0;
+        if (chem.ph.doserType === 2) schem.ph.chemType = 'CO2';
+        else if (chem.ph.doserType === 0) schem.ph.chemType = 'none';
+        else if (chem.ph.doserType === 1) schem.ph.chemType = 'acid';
+        else if (chem.ph.doserType === 3) schem.ph.chemType = 'acid';
+
+        if (chem.orp.doserType === 0) schem.orp.chemType = 'none';
+        else schem.orp.chemType = 'orp';
+
         schem.isActive = chem.isActive = true;
         schem.status = 0;
         schem.type = chem.type = sys.board.valueMaps.chemControllerTypes.getValue('intellichem');
@@ -156,8 +183,9 @@ export class IntelliChemStateMessage {
         schem.orp.volumeDosed = (msg.extractPayloadByte(18) * 256) + msg.extractPayloadByte(19);
         //      20 : pH tank level 1-7 = 6
         schem.ph.tank.level = Math.max(msg.extractPayloadByte(20) > 0 ? msg.extractPayloadByte(20) - 1 : msg.extractPayloadByte(20), 0); // values reported as 1-7; possibly 0 if no tank present
-        //      21 : ORP tank level 1-7 = 6
+        //      21 : ORP tank level 1-7 = 6 if the tank levels report 0 then the chemical side is not enabled.
         schem.orp.tank.level = Math.max(msg.extractPayloadByte(21) > 0 ? msg.extractPayloadByte(21) - 1 : msg.extractPayloadByte(21), 0); // values reported as 1-7; possibly 0 if no tank present
+
         //      22 : LSI = 3 & 0x80 === 0x80 ? (256 - 3) / -100 : 3/100 = .03
         let lsi = msg.extractPayloadByte(22);
         schem.lsi = (lsi & 0x80) === 0x80 ? (256 - lsi) / -100 : lsi / 100;
@@ -180,10 +208,26 @@ export class IntelliChemStateMessage {
         const alarms = schem.alarms;
         alarms.flow = msg.extractPayloadByte(32) & 0x01;
         if (alarms.flow === 0) schem.flowDetected = true;
-        alarms.pH = msg.extractPayloadByte(32) & 0x06;
-        alarms.orp = msg.extractPayloadByte(32) & 0x18;
-        alarms.pHTank = msg.extractPayloadByte(32) & 0x20;
-        alarms.orpTank = msg.extractPayloadByte(32) & 0x40;
+
+        // The pH and ORP alarms are in a word stupid for IntelliChem.  So we are
+        // going to override these.
+        //alarms.pH = msg.extractPayloadByte(32) & 0x06;
+        //alarms.orp = msg.extractPayloadByte(32) & 0x18;
+        if (chem.ph.tolerance.enabled && schem.flowDetected) {
+            if (schem.ph.level > chem.ph.tolerance.high) alarms.pH = 2;
+            else if (schem.ph.level < chem.ph.tolerance.low) alarms.pH = 4;
+            else alarms.pH = 0;
+        }
+        else alarms.pH = 0;
+        if (chem.orp.tolerance.enabled && schem.flowDetected) {
+            if (schem.orp.level > chem.orp.tolerance.high) alarms.orp = 8;
+            else if (schem.orp.level < chem.orp.tolerance.low) alarms.orp = 16;
+            else alarms.orp = 0;
+        }
+        else alarms.orp = 0;
+        // IntelliChem will still send a tank empty alarm even if there is no tank.
+        alarms.pHTank = (chem.ph.enabled && (chem.ph.doserType === 1 || chem.ph.doserType === 3)) ? msg.extractPayloadByte(32) & 0x20 : 0;
+        alarms.orpTank = (chem.orp.enabled && (chem.orp.doserType === 1 || chem.orp.doserType === 3)) ? msg.extractPayloadByte(32) & 0x40 : 0;
         alarms.probeFault = msg.extractPayloadByte(32) & 0x80;
         //      33 : Warnings -- pH Lockout, Daily Limit Reached, Invalid Setup, Chlorinator Comm error
         const warnings = schem.warnings;
@@ -206,7 +250,12 @@ export class IntelliChemStateMessage {
         //      36-37 : Firmware = 80,1 = 1.080
         chem.firmware = `${msg.extractPayloadByte(37)}.${msg.extractPayloadByte(36).toString().padStart(3, '0')}`
         //      38 : Water Chemistry Warning
-        schem.warnings.waterChemistry = msg.extractPayloadByte(38);
+        // The LSI handling is also stupid with IntelliChem so we are going to have our way with it.
+        // schem.warnings.waterChemistry = msg.extractPayloadByte(38);
+        schem.calculateSaturationIndex();
+        if (schem.saturationIndex > chem.lsiRange.high) schem.warnings.waterChemistry = 2;
+        else if (schem.saturationIndex < chem.lsiRange.low) schem.warnings.waterChemistry = 1;
+        else schem.warnings.waterChemistry = 0;
         if (typeof chem.body === 'undefined') chem.body = schem.body = 0;
         if (state.equipment.controllerType === 'nixie') {
             if (chem.ph.probe.feedBodyTemp) {
@@ -228,7 +277,6 @@ export class IntelliChemStateMessage {
         }
         schem.ph.pump.isDosing = schem.ph.dosingStatus === 0 && chem.ph.enabled;
         schem.orp.pump.isDosing = schem.orp.dosingStatus === 0 && chem.orp.enabled;
-        schem.calculateSaturationIndex();
         schem.lastComm = new Date().getTime();
         // If we are changing states lets set that up.
         if (schem.ph.dosingStatus === 0 && phPrev.status !== 0) {

package/controller/comms/messages/status/PumpStateMessage.ts

@@ -55,7 +55,29 @@ export class PumpStateMessage {
         state.emitEquipmentChanges();
         // if (msg.action !== 7) this.processDirectPumpMessages(msg);
     }
-
+    public static processHayward(msg: Inbound) {
+        switch (msg.action) {
+            case 12:
+                PumpStateMessage.processHaywardStar(msg);
+                break;
+        }
+    }
+    public static processHaywardStar(msg: Inbound) {
+        //             src   act   dest             
+        //[0x10, 0x02, 0x00, 0x0C, 0x00][0x00, 0x62, 0x17, 0x81][0x01, 0x18, 0x10, 0x03]
+        //[0x10, 0x02, 0x00, 0x0C, 0x00][0x00, 0x2D, 0x02, 0x36][0x00, 0x83, 0x10, 0x03] -- Response from pump
+        let ptype = sys.board.valueMaps.pumpTypes.transformByName('hwvs');
+        let pump = sys.pumps.find(elem => elem.address === msg.source + 96 && elem.type === 6);
+        if (typeof pump !== 'undefined') {
+            let pstate = state.pumps.getItemById(pump.id, true);
+            // 3450 * .5
+            pstate.rpm = Math.round(ptype.maxSpeed * (msg.extractPayloadByte(1) / 100));
+            // This is really goofy as the watts are actually the hex string from the two bytes.
+            pstate.watts = parseInt(msg.extractPayloadByte(2).toString(16) + msg.extractPayloadByte(3).toString(16), 10);
+            pstate.isActive = true;
+            state.emitEquipmentChanges();
+        }
+    }
     public static processDirectPumpMessages(msg: Inbound) {
 
         if (msg.payload.length === 2)

package/controller/nixie/Nixie.ts

@@ -118,11 +118,11 @@ export class NixieControlPanel implements INixieControlPanel {
     }
     public async closeAsync() {
         // Close all the associated equipment.
-        await this.pumps.closeAsync();
         await this.chemControllers.closeAsync();
         await this.chlorinators.closeAsync();
         await this.heaters.closeAsync();
         await this.circuits.closeAsync();
+        await this.pumps.closeAsync();
         await this.filters.closeAsync();
         await this.bodies.closeAsync();
         await this.valves.closeAsync();

package/controller/nixie/bodies/Body.ts

@@ -64,6 +64,9 @@ export class NixieBody extends NixieEquipment {
         super(ncp);
         this.body = body;
         this.pollEquipmentAsync();
+        let bs = state.temps.bodies.getItemById(body.id);
+        bs.heaterCooldownDelay = false;
+        bs.heatStatus = 0;
     }
     public get id(): number { return typeof this.body !== 'undefined' ? this.body.id : -1; }
     public async setBodyAsync(data: any) {