syssi / esphome-jk-bms

ESPHome component to monitor and control a Jikong Battery Management System (JK-BMS) via UART-TTL or BLE
Apache License 2.0
409 stars 137 forks source link

8 cells minimum is wrong #349

Closed pedro042 closed 1 year ago

pedro042 commented 1 year ago

Hi, I've BD4A8S4P which supports 4-8 cells. In settings is possible to configure min. 8 cells. It's wrong. Please don't limit count of cells to 8 or more. The BMS works well, I set count of cells in the Android app. Thanks.

syssi commented 1 year ago

Thanks for your report. I will push a fix!

pedro042 commented 1 year ago

Thanks; actually this BMS can handle 4 LifePo4 (minimum, because of the voltage needed to power the BMS) or 3 Li-Ion :-) (3 Li-Ion cells can power this BMS). Maybe one more limit decrease - 4 -> 3 https://www.aliexpress.com/item/1005004590744267.html - see the table, unfortunately as JPG picture only

syssi commented 1 year ago

Good catch! I will prepare another fix asap.

syssi commented 1 year ago

Fixed.