leftyfl1p / homebridge-poolcontroller

Homebridge plugin for Pentair pools
5 stars 5 forks source link

Temperature is not displayed when controller UOM is Celsius and Homekit is set to Celsius #10

Open gavc1 opened 4 years ago

gavc1 commented 4 years ago

For heating circuits, where the Intellitouch unit of measurement is Celsius, the poolTemp and spaTemp is reported to homekit as 0C.

When reading temperature, the UOM from the controller should be used, here is a sample of the JSON returned:

... "temperature": { "poolTemp": 17, "spaTemp": 17, "airTemp": 15, "solarTemp": 0, "freeze": 0, "poolSetPoint": 100, "poolHeatMode": 0, "poolHeatModeStr": "OFF", "spaSetPoint": 37, "spaManualHeatMode": "Off", "spaHeatMode": 0, "spaHeatModeStr": "OFF", "heaterActive": 0 }, "time": { "controllerTime": "9:02 PM", "controllerDateStr": "10/11/2019", "controllerDay": 11, "controllerMonth": 10, "controllerYear": 19, "controllerDayOfWeekStr": "Friday", "controllerDayOfWeek": 32, "automaticallyAdjustDST": 0, "pump1Time": -1, "pump2Time": -1 }, "UOM": { "UOM": 4, "UOMStr": "Celsius" }, ...