sfeakes / homebridge-aqualinkd

1 stars 1 forks source link

Salt Level PPM doesn't work with Celsius #17

Open talsalis opened 2 months ago

talsalis commented 2 months ago

The value is expressed in Fahrenheit and when user default is in Celsius, HomeKit "converts" it. Ex: 3200ppm shows as 1760C

Solution: Let the user indicate the temperature unit that should be used by default in the setup file so that it shows the "right value".

sfeakes commented 2 months ago

It's actually the other way round. But I get your point, and yes this needs to be done. HomeKit only accepts Celsius and will convert to Fahrenheit, or leave as is depending on phone setting. So since all phones I've come across so far are set to degF, I have to pass the PPM to HomeKit already converted knowing that HomeKit will automatically do a degF2degC conversion.

sfeakes commented 2 months ago

@talsalis Just checking your setup. The following should work correctly.

  1. Jandy panel is set to Celsius and your phone is also set to Celsius (AqualinkD will NOT convert PPM and will be displayed correctly)

  2. Jandy panel is set to Fahrenheit and your phone is also set to Fahrenheit (AqualinkD will convert PPM and will be displayed correctly, since HomeKit only accepts Celsius)

You are wanting a case when the Jandy panel is set to Fahrenheit but your phone is set to Celsius ?