search

esphome / issues

Issue Tracker for ESPHome
https://esphome.io/
292 stars 36 forks source link

Pylontech low limit on battery number #5351

Open nijel opened 10 months ago

nijel commented 10 months ago

The problem

US5000/US3000C/US2000C support 16 connected batteries, but this component only 6:

https://github.com/esphome/esphome/blob/fdd54d74a3ac8893fec8719760698dd3554cb16f/esphome/components/pylontech/__init__.py#L22

Which version of ESPHome has the issue?

2023.12.0

What type of installation are you using?

Home Assistant Add-on

Which version of Home Assistant has the issue?

2024.1

What platform are you using?

ESP32-IDF

Board

No response

Component causing the issue

pylontech

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

It seems that just the number needs to be increased.

nijel commented 10 months ago

Ah, this is addressed inside https://github.com/esphome/esphome/pull/6061 by @functionpointer (I didn't look inside PR until now and the title seemed unrelated to my issue).

github-actions[bot] commented 6 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.