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
310 stars 80 forks source link

Deactivate/ hide battery features #65

Closed i-peter closed 10 months ago

i-peter commented 1 year ago

Hi, first of all: Thanks for your great work!

I have a SH8.0RT and it's working great for me. Althought it was hard to find out, that I have to use the second LAN port next to the WiNet-S dongle to make it work. (BTW: Is there anywhere a guideline how to upgrade firmware for Sungrow? I only found outdated descriptions, which are not working for me.)

The question/ feature request I really have is the following: I don't have a battery and I would like to deactivate or hide all battery functionallities.

Would be great to have a central parameter instead of going through all lines of code to deactivate.

Best regards, Iven

mkaiser commented 1 year ago

hi

I have a SH8.0RT and it's working great for me. Althought it was hard to find out, that I have to use the second LAN port next to the WiNet-S dongle to make it work.

I have the first revision of the SH10.RT without WiNet-S, so I can't really say something about all this. I just know, that putting the cable in the WiNetS port often causes problems :/

Now, that you figured it out, could you give me some hints to update the documentation for this? I am pretty sure that would help others, too :)

I don't have a battery and I would like to deactivate or hide all battery functionallities. hmm the only way I see at the moment is to split the modbus_sungrow.yaml in multiple files, lets say "basic", "battery" and "history". I am afraid, that a more dynamic way is not possible in the yaml (there is no yaml-wide if/else possible).

Will give the splitting some thoughts in the future :)

With a more versatile Python integration for Home Assistant this would be much easier to achieve and more dynamic (just include feature XY, if checkbox is marked). Unfortunately, I have never used Python, yet and quiet frankly I don't have the time at the moment to learn it.

If someone knows how to do this, I would try to help :)

mkaiser commented 1 year ago

I am stuck trying to separate the config file. It would be nice if you can just put a .yaml file in a modbus_sungrow folder. If the yaml exists, it will be loaded. My tests ended up in crashes :/

mkaiser commented 10 months ago

had many thoughts about this.

I only have the capacity to deal with one integration variant (so the one with battery).

If you do not want them, delete the YAML entries.

I am sorry, but there is no other way with a pure YAML-integration. Only python could fix this