mkaiser / Sungrow-SHx-Inverter-Modbus-Home-Assistant

Sungrow SH Integration for Home Assistant for SH3K6, SH4K6, SH5K-20, SH5K-V13, SH3K6-30, SH4K6-30, SH5K-30, SH3.RS, SH3.6RS, SH4.0RS, SH5.0RS, SH6.0RS, SH5.0RT, SH6.0RT, SH8.0RT, SH10RT, SH5.0RT-20, SH6.0RT-20, SH8.0RT-20, SH10RT-20, SH5.0RT-V112, SH6.0RT-V112, SH8.0RT-V112, SH10RT-V112, SH5.0RT-V122, SH6.0RT-V122, SH8.0RT-V122, SH10RT-V122, SH4.6R
304 stars 79 forks source link

Battery temp from Battery #226

Closed kalaws closed 2 months ago

kalaws commented 6 months ago

Right now we are getting Battery temp from Inverter. However, at least in iSolarCloud battery temp is also reported by the battery. The difference is that Inverter gives no decimals, but rounds to integers. Batttery gives a precison of one decimal. Any way we could get this (hopefully through WiNet-S in my case, unfortunately)?

dylan09 commented 6 months ago

Scale for the sensor is defined as 0.1. So it should have decimals. But it seems the inverter only reports temperature without decimals. Also on my LAN port. But how accurate is the measured value? Does the precision of the measurement justify decimals?

kalaws commented 6 months ago

Yes, in iSolarCloud one can see 20,7, 20,8, 20,9 etc.

kalaws commented 6 months ago

And there is also in iSolarCloud the inverter value, which at the same time simply shows 21.

frankyman88 commented 6 months ago

My Battery temperature entity in home assistant reports one decimal place just as in ISolarCloud. In your modbus YAML make sure you have the following line in the battery temperature sensor config precision: 1