Replies: 143 comments 102 replies
-
Yeah eventually. Right now I'm focusing on getting another integration released (my tekmar-482 integration) and once that's out I'll revisit this one. I'm planning to rewrite this integration since I'm finding how it was written is limiting to things I want to do with it, and he structure I used for the tekmar-482 integration is more flexible for me. Edit: batteries are supported starting in v2.0.0 and StorEdge control starting with v2.2.7 |
Beta Was this translation helpful? Give feedback.
-
+1 from me! |
Beta Was this translation helpful? Give feedback.
-
The rewrite-for-2.0 branch is at a point where it should detect batteries and create device entries for them, but there aren't any sensor entities yet. |
Beta Was this translation helpful? Give feedback.
-
Anyone who has batteries please try the rewrite-for-2.0 branch and report if it detects your batteries or not. It will have some basic diagnostic sensors and show a device for each battery (up to 2 per inverter for up to 32 inverters). Because battery registers are not technically officially supported by SolarEdge, auto-detection for batteries is disabled by default. To enabled battery detection, go to the device options and enable it. |
Beta Was this translation helpful? Give feedback.
-
Hi, I gave this a shot, I got devices for the inverter and the 2 meters, but no batteries. I have the newer SolarEdge Battery Banks BAT-10K1P, there are two of them. I rolled back to the normal version for now since I wasn't sure if all of the entity names changed. I'll need to go edit a bunch of stuff to keep things working. |
Beta Was this translation helpful? Give feedback.
-
Inverter entity names will be the same, meter entities will be slightly different since auto detect will scan every inverter for all 3 meters, which means it's possible to have more than 3 meters, so it names the meter entities with the inverter ID and the meter ID instead of fixed numbers. (I should say the inverter entity names are supposed to be unchanged but I haven't double checked them in old vs. new yet so there could be mistakes since this is still pre-pre release). If you set debug logging it will log something for batteries. Basically the way auto detect works is it tries to read some basic info registers and if the read is OK then it adds the device. If the read returns any kind of error it ignores that location. The battery registers have been taken from an engineering pdf for the LG batteries where only 2 batteries per inverter were supported. To enable debug logging:
And then in my logs the important lines are:
Which means there aren't any batteries at the battery register, in my case because I don't have any so the address doesn't exist. If you see those exceptions and you're not using the old LG batteries, then that means it's likely SolarEdge changed the battery registers or isn't using them. |
Beta Was this translation helpful? Give feedback.
-
I just pushed some updates to logging attributions so you'll need to re-download. |
Beta Was this translation helpful? Give feedback.
-
Hmm yep looks like the new ones aren't using the registers... That's very unfortunate...
|
Beta Was this translation helpful? Give feedback.
-
I assume you're using the new "Energy Bank" battery system? Edit: you already said that you are |
Beta Was this translation helpful? Give feedback.
-
Well, I'll keep an eye out for any information about registers for the new Energy Bank system. Otherwise it's anyone's guess if they moved to different addresses or what. |
Beta Was this translation helpful? Give feedback.
-
Yep, no worries. Hopefully somewhere down the road. Thanks! |
Beta Was this translation helpful? Give feedback.
-
It's still possible I got the registers wrong since I can't test any battery stuff myself, hopefully someone with the older battery system will give it a test to confirm it's an LG vs Energy Bank difference. |
Beta Was this translation helpful? Give feedback.
-
Pre-release v2.0.0-pre.1 is available now. |
Beta Was this translation helpful? Give feedback.
-
I have two Inverter and each inverter has one battery attached. But I do not think SolarEdge is exposing the batteries via Modbus TCP at all. Using modbus test tools, there are no values exported where they should be according to the SunSpec. I guess the inverters are collecting the battery info via RS485 but do not copy the values into the Modbus TCP range. Luckily the LG batteries have a mini WebServer on board that can easily be used to read all BMS values |
Beta Was this translation helpful? Give feedback.
-
LG batteries should be detected. My code may have been off by one count on the battery info read block, I will release a pre.7 with what I think might be the correct value. |
Beta Was this translation helpful? Give feedback.
-
I'm looking for anyone with batteries who can test Release v2.2.5-pre.3 which contains new validation rules for battery sensor values. Edit: v2.2.5-pre.2 contains a load error, use v2.2.5-pre.3 |
Beta Was this translation helpful? Give feedback.
-
Tell me what to look for, logs, etc....
…On Sun, Oct 9, 2022, 16:47 Seth ***@***.***> wrote:
I'm looking for anyone with batteries who can test Release v2.2.5-pre.2
<https://github.com/WillCodeForCats/solaredge-modbus-multi/releases/tag/v2.2.5-pre.2>
which contains new validation rules for battery sensor values.
—
Reply to this email directly, view it on GitHub
<#63 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJ4FIEBRUIFNVXF6C5GCGLWCLEHXANCNFSM55CBRHIQ>
.
You are receiving this because you were mentioned.Message ID:
<WillCodeForCats/solaredge-modbus-multi/repo-discussions/63/comments/3833218
@github.com>
|
Beta Was this translation helpful? Give feedback.
-
Hi, I can't get the right information from battery: unknown or 0 are returned. |
Beta Was this translation helpful? Give feedback.
-
I'm also looking to integrate my system into my smart house (PLC based) but i can't find any doc on what the modbus registers are for the solaredge energy bank. Can anyone help me out with that? |
Beta Was this translation helpful? Give feedback.
-
So I just installed this integration and it works great except for the batteries. I have 3 Solaredge 7600H inverters, a backup interface (basically an energymeter/transfer switch) and 3 LG RESU16H Prime batteries. I'm getting the invalid address error in the debug log for the batteries. |
Beta Was this translation helpful? Give feedback.
-
Does anyone have have a Generator interface on the Backup Interface of SolarEdge? I had this installed with my initial installation back in 2021 and a generator is supposed to charge the batteries at times when the power is out and the batteries are low. |
Beta Was this translation helpful? Give feedback.
-
If anyone has three batteries, please try manually installing this: https://github.com/WillCodeForCats/solaredge-modbus-multi/archive/refs/tags/b3-test-1.zip It's a blind read test based on the existing battery for 2 batteries, and if that doesn't work I an try the "battery 1 mirror" location too. Mirror b1 address for b3: https://github.com/WillCodeForCats/solaredge-modbus-multi/archive/refs/tags/b3-test-2.zip |
Beta Was this translation helpful? Give feedback.
-
Does anybody have what, exactly, they asked solaredge to do in order to enable the modbus power control/battery registers? I scanned all my registers and it's pretty clear that the battery ones aren't enabled. I sent this to support:
And I got this back:
Obviously this person has no idea what I'm talking about, but I'm hoping someone here has had some luck asking for a particular department or person or something. Thanks! |
Beta Was this translation helpful? Give feedback.
-
Ping, if there are any other tests I can do to help identify the integration of battery 3? |
Beta Was this translation helpful? Give feedback.
-
If I had a statement their devs would get, I will lodge a product request?
On Saturday, April 1, 2023, 2:00 am, Seth ***@***.***> wrote:
Probably not. The modbus spec only defines 2 batteries, so any test I were doing were just "see what would happen" attempts, nothing I seriously expected to work. Unless SolarEdge revises their modbus support to define more batteries two will remain the limit.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
It's been reported in discussion #233 modbus may now support three batteries. Please try Release v2.4.9-pre.2 if you have a third battery. |
Beta Was this translation helpful? Give feedback.
-
I have 2 SE batteries and a SE10000H-US (my second one due to failure on
the first).
I wish I had news to share but I have nothing.
I keep hoping like everyone else in this situation.
…On Thu, Feb 29, 2024, 5:23 PM brde21 ***@***.***> wrote:
@jeremy-asher <https://github.com/jeremy-asher> Yes, I do have LG
batteries. I don't think it would have anything to do with the battery
manufacturer though. It appears, from what I understand, to be a setting or
access permission in the inverter firmware itself. I would like to see the
SetApp screens from an inverter that the battery access has been enabled on
as well. Perhaps it would offer some clue. I keep hoping @rtlinux
<https://github.com/rtlinux> or @bdimcheff <https://github.com/bdimcheff>
or @cpaquette63 <https://github.com/cpaquette63> would see these posts or
an auto-email notification and chime in, but no luck with that so far
either.
—
Reply to this email directly, view it on GitHub
<#63 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A4T7HBBEAWDHD7CZ6P3ZUPTYV6U6DAVCNFSM55CBRHI2U5DIOJSWCZC7NNSXTOKENFZWG5LTONUW63SDN5WW2ZLOOQ5TQNRTGYZTGMY>
.
You are receiving this because you were mentioned.Message ID:
<WillCodeForCats/solaredge-modbus-multi/repo-discussions/63/comments/8636333
@github.com>
|
Beta Was this translation helpful? Give feedback.
-
I now see the third battery, yayh.
|
Beta Was this translation helpful? Give feedback.
-
I am also seeing a third battery, now it is all connected up :) Can you add a known issue to ensure people are aware that the Battery Firmware Version is truncated and so doesn't show the BMS value correctly. The firmware of the battery is reported by the SolarEdge Integration as an attribute to sensor.solaredge_batteries which has the following text for the first battery in my system:
In this integration, this gets reported in sensor.solaredgemodbus_b1_version as:
The last two digits (51) are missing You can see the same issue present in cralwalker's report above, the BMS version is missing at least one digit after the period Given there are 32 characters, it seemed suspiciously like there's a hard limit and reading through the spec for Power Control on Page 21 it does seem that the return is a 32 character string, so I assume there's nothing that can be done.
Debug shows the value exactly the same as reported, so I think known issue is all that can be done as I can't see anywhere else to get the value from:
|
Beta Was this translation helpful? Give feedback.
-
Does it have anything to do with Power Control settings? Some (old) documentation Power Control Open Protocol (photovoltaikforum.com)<https://www.photovoltaikforum.com/core/attachment/88445-power-control-open-protocol-for-solaredge-inverters-pdf/> seems to imply that if you dont have these set it wont work: To enable the dynamic power control mode, set the following: AdvancedPwrControlEn on address 0xF142 to 1 (enable). It is disabled (0) by default. ReactivePwrConfig on address 0xF104 to 4 (RRCR mode). It is set by default to 0 (Fixed CosPhi mode). Issue a Commit Power Control Settings command on address 0xF100 in order to make this setting effective
That said, I dont even seem to be able to read those registers to begin with. I set what i think were the right ones in setapp, but nothing changed.
…________________________________
From: brde21 ***@***.***>
Sent: Friday, March 29, 2024 11:58 AM
To: WillCodeForCats/solaredge-modbus-multi ***@***.***>
Cc: mikedeluca ***@***.***>; Mention ***@***.***>
Subject: Re: [WillCodeForCats/solaredge-modbus-multi] Battery integration (Discussion #63)
Okay, that's the same version I am on so it's not directly firmware related. I guess SolarEdge is just being a bunch of clueless a-holes towards me.
—
Reply to this email directly, view it on GitHub<#63 (reply in thread)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AEPAANQM4BJEUPTJA5VLV7LY2W2VVAVCNFSM55CBRHI2U5DIOJSWCZC7NNSXTOKENFZWG5LTONUW63SDN5WW2ZLOOQ5TQOJVGQ4TKNI>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
Are there any plans to integrate connected batteries?
Beta Was this translation helpful? Give feedback.
All reactions