screenlogic/node-screenlogic
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

GetPumpStatus isRPMs changed to bool to be consistent with SetPumpStatus

Browse Source
This commit is contained in:
Chris Pickett
2020-06-09 08:42:26 -05:00
parent bf3902efe8
commit ee30cd815e
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
README.md
View File

@ -624,4 +624,4 @@ Passed as an argument to the emitted `setPumpFlow` event. The passed version is
* `pumpId` - id of pump to get information about, first pump is 0
* `circuitId` - index of circuit for which to change the set point (index is relative to data returned by [`SLGetPumpStatus`](#slgetpumpstatus))
* `setPoint` - the value for which to set the pump/circuit combo
* `isRPMs` - boolean, TRUE for RPMs, FALSE for GPMs
* `isRPMs` - boolean, `true` for RPMs, `false` for GPMs

2
messages/SLGetPumpStatus.js
View File

@ -40,7 +40,7 @@ exports.SLGetPumpStatus = class SLGetPumpStatus extends SLMessage {
this.pumpSetting[i] = {};
this.pumpSetting[i].circuitId = this.readUInt32LE();
this.pumpSetting[i].pumpSetPoint = this.readUInt32LE();
this.pumpSetting[i].isRPMs = this.readUInt32LE(); // 1 for RPMs; 0 for GPMs
this.pumpSetting[i].isRPMs = this.readUInt32LE() !== 0; // 1 for RPMs; 0 for GPMs
}
}