search

Michsior14 / ha-venta

Home assistant integration for Venta devices
MIT License
11 stars 6 forks source link

Venta LPH60 shows fahrenheit temperature instead of celsius #57

Closed klim123123 closed 1 week ago

klim123123 commented 1 week ago

In Home Assistant it's show the digits in Fahrenheit, but symbol is C. 78.9 F is exactly 26.1 C how to switch to Celsius digits in Home Assistant ? Thank you

image image

Michsior14 commented 1 week ago

Could you please enable the debug mode for integration wait for one refresh and attach the log?

I assume this is possible if the device has Fahrenheit set in the parameter store, but would need the confirmation.

klim123123 commented 1 week ago 
{
  "Header": {
    "DeviceType": 2,
    "MacAdress": "xx:f0:05:xx:xx:xx",
    "Hash": "-42",
    "Error": 0,
    "DeviceName": ""
  },
  "Action": {
    "Power": true,
    "FanSpeed": 1,
    "TargetHum": 70,
    "Timer": 0,
    "Boost": false,
    "SleepMode": false,
    "ChildLock": false,
    "Automatic": false,
    "SysLanguage": 7,
    "CleanLanguage": 2,
    "TempUnit": 1,
    "DisplayLeft": 0,
    "DisplayRight": 0,
    "Reset": 0,
    "ConINet": 0,
    "DelUser": false
  },
  "Info": {
    "SWDisplay": "V2.013",
    "SWPower": "V1.001",
    "SWTouch": "V1.3.0",
    "SWWIFI": "V19.4.8",
    "CleanMode": false,
    "RelState": [false, false, false, false],
    "TimerT": 0,
    "OperationT": 243326,
    "DiscIonT": 203,
    "CleaningT": 232,
    "FilterT": 99,
    "UVCOnT": 0,
    "UVCOffT": 0,
    "CleaningR": 0,
    "Warnings": 1
  },
  "Measure": {
    "Temperature": 79.1,
    "Humidity": 44.7,
    "Dust": 1,
    "WaterLevel": 2,
    "FanRpm": 0
  }
}
Michsior14 commented 1 week ago

Yeah, as suspected temperature unit is set as imperial:

  {
    "name": "TempUnit",
    "type": "integer",
    "description": "Display unit of temperature.",
    "values": [
      {
        "value": 0,
        "type": "integer",
        "description": "Metric"
      },
      {
        "value": 1,
        "type": "integer",
        "description": "Imperical"
      }
    ]
  }

I think we can make it work by always converting to Celcius in this case / or setting different unit during the native value set. Let me check how it works as there were some changes in this area recently on HA side.

Michsior14 commented 1 week ago

Please check the v0.10.0 release.

klim123123 commented 1 week ago

Thank you. Is it how it suppose to be? If i need Celciuse, do i need to convert myself?

image

Michsior14 commented 1 week ago

Nope, I've used the wrong scope for the TempUnit, should be fixed in the next 5 minutes :)

Michsior14 commented 1 week ago

Please retest

klim123123 commented 1 week ago

Before updating, I just noticed, i have this. image

klim123123 commented 1 week ago

After update, all good. Thank you. image