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
344 stars 92 forks source link

Tiny visual bug at temperature sensors units #320

Closed Tabis83 closed 2 months ago

Tabis83 commented 2 months ago

Before you create an issue, make sure to update to the current version of modbus_sungrow.yaml

Describe the bug:

Tiny visual bug at temperature sensors units. Instead of [float] °C, it shows [float] °C

Your Sungrow inverter:

Home Assistant version:

modbus_sungrow.yaml:

Inverter Firmware Status:

To Reproduce Steps to reproduce the behavior:

  1. Ceck one of the temperature sensors like "Inverter temperature" or "Battery temperature" in the latest version of the PV Dashboard.

Expected behavior Get rid of the additional "Â" in front of the unit.

Screenshots Bildschirmfoto vom 2024-07-27 11-10-34

Additional context Add any other context about the problem here.

Gnarfoz commented 2 months ago

That looks more like an encoding issue on your end. How did you download the file? Have you checked if a different web browser also shows it like this?

Tabis83 commented 2 months ago

Hi, Thanks you are right. 👍 I've created a new file in the integrations folder and copied the content from GitHub file to the new file via clipboard using an iOS device. This probably caused the encoding error. The sensor unit in the modbus_sungrow.yaml was "°C" 🙈

After changing and restart of Home Assistant everything is fine.