search

kellerza / sunsynk

Deye/Sunsynk Inverter Python library and Home Assistant OS Addon
https://kellerza.github.io/sunsynk/
MIT License
229 stars 95 forks source link

max limit for max charge and discharge current #167

Open trackmafia opened 1 year ago

trackmafia commented 1 year ago

Existing sensor definitions can be found here

You Inverter

Sunsynk 16kw

Describe the issue/bug and what you expect

using this addon and it is working perfectly with 16kw inverter using your instructions, well done, much appreciated.

For sensor battery max charge current and max discharge current, max is set to 185. However on 16k inverter max can go up to 275 for both. this is for modbus register 210 and 211. I created a custom sensor to get around this. However, it shows up as a number and not a slider and i'm not sure how to change this in home assistant? Can you advise if the max can be changed or how to make the custom sensor show up with a slider as in the original?

Also, please note that 16kw has 3 MPPTs, i got around this easily by creating custom sensors for pv3_power, pv3_current and pv3_volts . Just posting here for anybody in the future.

Logs (if applicable)

kellerza commented 1 year ago

Thanks @trackmafia - try NUMBER_ENTITY_MODE for the slider - see here. The default auto mode might have decided that 285 is too much for a slider.

Do you mind doing a PR to add the 3rd MPPT? It should be ok to change Max current as well.

trackmafia commented 1 year ago

Kellerza,

This worked perferctly. Thanks very much. I'm new to github but i'll learn how to do pull request and submit within a week.

Thanks again for this awesome software.