esphome / issues

Issue Tracker for ESPHome
https://esphome.io/
290 stars 34 forks source link

Pylontech component gives wrong unit °C instead of Voltages #5271

Open neujbit opened 9 months ago

neujbit commented 9 months ago

The problem

I have 3 Pylontech US5000C and add an ESPmini to read them out. All working fine but the units from voltage_low and voltage_high showing in °C and have the wrong values. I think the units problem is in Sensors init_py: CONF_VOLTAGE_LOW: sensor.sensor_schema( unit_of_measurement=UNIT_CELSIUS, accuracy_decimals=1, device_class=DEVICE_CLASS_TEMPERATURE, ), CONF_VOLTAGE_HIGH: sensor.sensor_schema( unit_of_measurement=UNIT_CELSIUS, accuracy_decimals=1, device_class=DEVICE_CLASS_TEMPERATURE,

Which version of ESPHome has the issue?

2023.12.2

What type of installation are you using?

Home Assistant Add-on

Which version of Home Assistant has the issue?

2023.12.3

What platform are you using?

ESP8266

Board

ESP01s

Component causing the issue

Pylontech

Example YAML snippet

No response

Anything in the logs that might be useful for us?

[10:28:46][D][sensor:093]: 'Battery1 lowest Voltage': Sending state 3.24000 °C with 1 decimals of accuracy
[10:28:46][D][sensor:093]: 'Battery1 highest Voltage': Sending state 3.24600 °C with 1 decimals of accuracy

[W][pylontech:079]: invalid line: found only 1 items in 16    -      -      -      -      -      -      -      Absent   -        -        -        -        -                    -        -       
Command completed successfully
$$

Additional information

No response

nijel commented 8 months ago

https://github.com/esphome/esphome/pull/6060 by @functionpointer fixes this.