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

Add information of EV charger (for car) to available values #242

Open GolfOWL opened 8 months ago

GolfOWL commented 8 months ago

First I want to thank all of you, especially mkaiser, for this great add-on in HA. I have had a PV system since yesterday and the add-on has made it very easy for me to integrate it into HA.

In addition to the inverter and the battery, I also have a wallbox from Sungrow, which is connected and controlled via the management system.

As a result, I can imagine that corresponding parameters are also available for the "EV Charger1", which can be integrated into HA.

I would like to address such an extension with this feature request.

I would like to thank you in advance for your feedback and hope that I have done this "correctly" in the form of this request.

andi-blafasl commented 1 week ago

I have ordered the Wallbox yesterday and also want to integrate it into homeassisant via modbus. Found something about the registers in the iobroker forum. Maybe I get the wallbox up and running next weekend. Have some challenges with cabling from my house to the garage ;-) Hopefully the RS485 over Ethernet adapter will save me from pulling a new cable...

lassej01 commented 1 week ago

I have ordered the Wallbox yesterday and also want to integrate it into homeassisant via modbus. Found something about the registers in the iobroker forum. Maybe I get the wallbox up and running next weekend. Have some challenges with cabling from my house to the garage ;-) Hopefully the RS485 over Ethernet adapter will save me from pulling a new cable...

Maybe you want it all local, but otherwise, you can use this already existing integration for wallbox in Home assistant: https://www.home-assistant.io/integrations/wallbox/

Gnarfoz commented 1 week ago

I doubt that works with the Sungrow AC011E-01 wallbox.

Louisbertelsmann commented 1 week ago

How about this? Disclaimer: I don’t have a wallbox, but other people have debugged it and it should work. https://github.com/Louisbertelsmann/Sungrow-Wallbox-Modbus-HomeAssistant

Louisbertelsmann commented 1 week ago

If you want to read the wallbox through the inverter, you should be able to copy the sensors into the sungrow_modbus.yaml because sungrow doesn’t like it if two modbus hubs are on the same IP.

andi-blafasl commented 1 week ago

Louis, that's a great find! Bit I think you do not need to integrate it with the inverter yaml. They are reading the wall box data via winet-s dongle not via the integrated lan port. So no two hubs accessing the inverter directly. Will try it as soon as possible.

Gnarfoz commented 1 week ago

That should work regardless of if you talk to the WiNet-S or directly to the inverter, I think.

lassej01 commented 1 week ago

I doubt that works with the Sungrow AC011E-01 wallbox.

No, totally with you on that one. For me a Wallbox is an own brand, I wasnt aware that Sungrow called their Wallbox, Wallbox. And the person who asked didnt state it was a Sungrow.

Louisbertelsmann commented 1 week ago

That should work regardless of if you talk to the WiNet-S or directly to the inverter, I think.

Unfortunately not. You can’t have two hubs on one single IP AFAIK. Homeassistant doesn’t like that.

Gnarfoz commented 1 week ago

No, totally with you on that one. For me a Wallbox is an own brand, I wasnt aware that Sungrow called their Wallbox, Wallbox. And the person who asked didnt state it was a Sungrow.

The link they included pointed to a german forum thread about the Sungrow product, though. Apparently it's a language thing. In German, "Wallbox" has been referring to home charging stations for far longer than the Spanish company "Wallbox" has been around. Looks like it's one of those words that look like they're English loan words but they're really not. (Another common example: "Handy", meaning mobile phone, is not actually an English noun at all).

Unfortunately not. You can’t have two hubs on one single IP AFAIK. Homeassistant doesn’t like that.

Why use two hubs, though? I use a single hub (pointing to the inverter LAN port), and some sensors use slave id 1 (inverter) and some use slave id 200 (battery). Shouldn't adding the sensors to the same hub and using the appropriate slave id (2?) work the same?

Louisbertelsmann commented 1 week ago

Unfortunately not. You can’t have two hubs on one single IP AFAIK. Homeassistant doesn’t like that.

Why use two hubs, though? I use a single hub (pointing to the inverter LAN port), and some sensors use slave id 1 (inverter) and some use slave id 200 (battery). Shouldn't adding the sensors to the same hub and using the appropriate slave id (2?) work the same?

That’s what I already proposed earlier. I just said that it won’t work out-of-the-box when just copy pasting that into another yaml file. You will have to either copy the sensors into the existing one or use another IP to talk to the wallbox, either directly or through the inverter.

Gnarfoz commented 1 week ago

Sorry, I must have misread that. :-)

andi-blafasl commented 4 days ago

I got my Charger installed yesterday and followed the instructions from https://github.com/Louisbertelsmann/Sungrow-Wallbox-Modbus-HomeAssistant to setup up home assistant. For now, I see values but can't test any charging because the lag of an EV 😁