search

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
328 stars 86 forks source link

Update modbus_sungrow.yaml #26

Closed andi-blafasl closed 1 year ago

andi-blafasl commented 1 year ago

Fixed Typo Phase X Current unit_of_measurement FIxed Scale Grid Frequency

mkaiser commented 1 year ago

hi,

thank you for the fix!

I can't directly merge this PR, because, the frequency was working for me fine before (with scale: 0.1) and my SH10.RT. What inverter are you using requiring the 0.01 scaling?

mkaiser commented 1 year ago

fixed in https://github.com/mkaiser/Sungrow-SHx-Inverter-Modbus-Home-Assistant/commit/18ce7eb873cdc83581fd0a5f467c3d18d9158016 - thank your for reporting.

See https://github.com/mkaiser/Sungrow-SHx-Inverter-Modbus-Home-Assistant/issues/34#issuecomment-1352885316, if the grid frequency is still off