crsherman / nodejs-poolController-mqtt

A MQTT integration for nodejs-poolController from tagyoureit
GNU General Public License v3.0
7 stars 4 forks source link

water temps (pool and spa) are not published #1

Closed gavc1 closed 4 years ago

gavc1 commented 5 years ago

Testing this plugin I get the following topics published pool/circuit/ pool/poolheat/ pool/spaheat/

Within the topics above I can see the mode and set temp are published, but the current temperature is not. I do see it in the console log outputSocketToMQTT: Temperature info as follows: {"temperature":{"poolTemp":35,"spaTemp":35,"airTemp":32,"solarTemp":0,"freeze":0,"poolSetPoint":26,"poolHeatMode":0,"poolHeatModeStr":"OFF","spaSetPoint":35,"spaManualHeatMode":"Off","spaHeatMode":1,"spaHeatModeStr":"Heater","heaterActive":1}}

But cannot find anywhere in outputSocketToMQTT.js where you publish the temperature

crsherman commented 5 years ago

Gavc1, Since I mainly use MQTT for the ability to control my pool equipment from node-red, I am not publishing the temperatures (air and water) using my plugin. I use the nodejs-poolController api to request the temperatures for air and water to feed into my node-red graph. See the readme for nodejs-poolController - APIs section for more info.

If you are still interested in a MQTT solution, I could easily add the code to it when I have a few minutes to get to it. Let me know if you are still interested in lieu of the API.

On Oct 29, 2019, at 11:18 PM, gavc1 notifications@github.com wrote:

Testing this plugin I get the following topics published pool/circuit/ pool/poolheat/ pool/spaheat/

Within the topics above I can see the mode and set temp are published, but the current temperature is not. I do see it in the console log outputSocketToMQTT: Temperature info as follows: {"temperature":{"poolTemp":35,"spaTemp":35,"airTemp":32,"solarTemp":0,"freeze":0,"poolSetPoint":26,"poolHeatMode":0,"poolHeatModeStr":"OFF","spaSetPoint":35,"spaManualHeatMode":"Off","spaHeatMode":1,"spaHeatModeStr":"Heater","heaterActive":1}}

But cannot find anywhere in outputSocketToMQTT.js where you publish the temperature

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/crsherman/nodejs-poolController-mqtt/issues/1?email_source=notifications&email_token=ABABZYAIW4IXIXX3WXJINYLQREDIZA5CNFSM4JGTH3DKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HVI6TXA, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABABZYCUSXXXL74JGDAKMNTQREDIZANCNFSM4JGTH3DA.

crsherman commented 4 years ago

Issue closed with version 1.1.