kellerza / sunsynk

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

Missing Sunsynk Inverter and Battery Temperature #195

Closed ai001 closed 8 months ago

ai001 commented 1 year ago

Issue related to the Python sunsynk library

Describe the issue/bug

Not really a bug but a request to make available temperature values of various component from the Sunsynk inverters where it can be used to trigger additional cooling fan on and off.

Expected behavior

A request to make available temperature values of various component from the Sunsynk inverters where it can be used to trigger additional cooling fan on and off.

TomBeech85 commented 1 year ago

I came here to raise this feature request. I want it to make sure the I can get an alert if the batteries get too cold during the winter months

iBeech commented 1 year ago

Issue related to the Python sunsynk library

Describe the issue/bug

Not really a bug but a request to make available temperature values of various component from the Sunsynk inverters where it can be used to trigger additional cooling fan on and off.

Expected behavior

A request to make available temperature values of various component from the Sunsynk inverters where it can be used to trigger additional cooling fan on and off.

You can add two sensors to achieve this:

kellerza commented 1 year ago

In the past I've tried adding more temperature sensors, without luck

You can experiment with new sensors/registers using custom sensors - see https://kellerza.github.io/sunsynk/reference/mysensors

steve-prentice commented 11 months ago

Also here to ask the same...

battery_temp_182: sensor.ss_battery_temperature

Is mentioned in the example template: https://kellerza.github.io/sunsynk/examples/templates

But doesn't seem to exist, I can't see it in 'Sunsynk/Deye Inverter Add-on (multi)' logs as a sensor that's being read?

I've read the Custom Sensors doc, but it blows my mind as a newbie, and I'd hope that it should be there without needing to add it as custom in any case?

I'd also like to get inverter_voltage_154: sensor.ss_inverter_voltage which is missing as well as far as I can see?

And sensor.ss_grid_current is also mentioned in the template and isn't collected from what I can see.

Thanks for your help! :-)

steve-prentice commented 11 months ago

I've read the documentation in way more depth now than I think was needed for this... so if anyone else is wanting battery temp and not finding it easy, edit your Configuration tab in HA - I chose to Edit in YAML - and add to your Sensors area, for example I added this:

SENSORS: