wills106 / homeassistant-solax-modbus

SolaX Power Modbus custom_component for Home Assistant (Supports some Ginlong Solis, Growatt, Sofar Solar, TIGO TSI & Qcells Q.Volt Hyb)
310 stars 95 forks source link

[FR]: Growatt MODxxKTL3-XH - Add missing battery entities #757

Open iobrokerneu opened 6 months ago

iobrokerneu commented 6 months ago

Describe the bug

hi, would it be possible to create an entity from the battery capacity in %.

I use a Growatt Mod 5000TL3-XH with an ARK HV battery.

Thanks and greetings Sascha

Integration Version

2024.01.1

Homeassistant core version

2024.2.1

Inverter brand

Growatt

Plugin used

plugin_growatt.py

Serial prefix

JDM0DB

Connection Method

USR TCP232-410S

Detailed Error Log

No response

Additional context

No response

wills106 commented 6 months ago

I have just released 2024.02.6b1 with some more battery entities set to Gen4 as the MOD seems to use a mix of Gen2 and Gen4. But it's not documented which are used.

iobrokerneu commented 6 months ago

I cannot install the new beta version. It's displayed, but you can't install it.

iobrokerneu commented 6 months ago

IMG_7363

wills106 commented 6 months ago

Can't you select 2024.02.6b1 from that drop down? Or won't it let you?

iobrokerneu commented 6 months ago

Whenever I select 6b1, it always jumps to 2.5. Could it perhaps be because it is in beta below 2.5? I've already tested it on my iphone and PC.

wills106 commented 6 months ago

If you go to the HACS page, Integrations, click on "SolaX Inverter Modbus" then the 3 dots top right. Then Redownload. Can you select 2024.02.6b1 that way?

Just updated for me. image

iobrokerneu commented 6 months ago

Unfortunately does not work . ha was also restarted.

https://github.com/wills106/homeassistant-solax-modbus/assets/55629670/176ba68e-1b03-448b-a1ba-9fca0cb8b111

iobrokerneu commented 6 months ago

hi, is this a general problem or is it me? greeting

wills106 commented 6 months ago

Just pushed out 2024.02.6b2 as there was an update for Sofar. Does that show up now?

image

iobrokerneu commented 6 months ago

hi, I have just installed the new beta and the battery display is displayed. Very good work. Thanks and greetings Sascha IMG_7379

wills106 commented 6 months ago

Do the other Battery sensors work or display correctly?

iobrokerneu commented 6 months ago

generally all data is displayed correctly. The above data is currently not available. But you set it that way. The process can be closed again immediately. Thanks IMG_7382

wills106 commented 6 months ago

AC Power to x can be deleted if you click on each one that says Unknown.

Are you able to control the battery from the Integration? Or does that need work still?

Edit: Looks like there aren't any battery control entities exposed for MOD Hybrid. Probably should look into that and also this comment onwards https://github.com/wills106/homeassistant-solax-modbus/issues/735#issuecomment-1952693523 so we can determine if a MOD is a Hybrid or PV only Inverter.

iobrokerneu commented 6 months ago

I don't control the battery directly. I just read the data via modbus. Unfortunately I can't test that.

wills106 commented 6 months ago

I'm going to leave this open for now incase anyone else wants' to feedback on extra battery entities for the MOD.

Emilthn commented 5 months ago

@wills106 I've got a Hybrid MOD inverter with a ARX battery and I can see all values but I can't change "Priority" from "Load First", when trying it always jumps back to "Load First". So from what I've been reading you must write start and stop time to holding registers 3038 and 3039 because Growatt have chosen to merge mode selection with time slots. Changing only 3144 doesn't work.

Information found here: https://community.home-assistant.io/t/esphome-modbus-growatt-shinewifi-s/369171/323

wills106 commented 5 months ago

Growatt have chosen to merge mode selection with time slots.

It hurts my head just looking at it! Which bits do you want to control?

Emilthn commented 5 months ago

Haha yes, it's a "bit" messy..! For my MOD 10KTL3-XH changing only 3144 doesn't work but I don't really understand if register 3144 also should be set at the same time as registers 3038 and 3039.. I guess it's best to just do a test and see if it works by just setting 3038/3039.

So he is using these values for setting the different modes. "Load First" Register 3038 = 32768 Register 3039 = 5947

" Battery First" Register 3038 = 40960 Register 3039 = 5947

"Grid First" Register 3038 = 49152 Register 3039 = 5947

Does it make sense for you?

wills106 commented 5 months ago

Ahh that's ok then. I just wanted to make sure you didn't want to adjust times as well, as that would have been a nightmare!

JohNan commented 3 months ago

This would be great if implemented. The proposed value does however implicitly also set the start and end time to 00:00 - 23:59. That means that it will fail to write if any of the other 8 time slots are enabled and have an overlapping time.

But it a good start and will have the desired effect of choosing mode.

I can confirm that it it's working, i wrote these values manually using another modus client to my MOD 6ktl3-xh

Here's all the registers and the bits

Screenshot_20240610_211930_Drive.jpg

abspoel99 commented 1 month ago

Haha yes, it's a "bit" messy..! For my MOD 10KTL3-XH changing only 3144 doesn't work but I don't really understand if register 3144 also should be set at the same time as registers 3038 and 3039.. I guess it's best to just do a test and see if it works by just setting 3038/3039.

So he is using these values for setting the different modes. "Load First" Register 3038 = 32768 Register 3039 = 5947

" Battery First" Register 3038 = 40960 Register 3039 = 5947

"Grid First" Register 3038 = 49152 Register 3039 = 5947

Does it make sense for you?

Hello, would love to edit the existing code to make this work on my inverter. But as im not familiar with coding / yaml I’m struggling with making the correct changes to test this.

Would any of you be so kind to give me some pointers Do i need to add a extra entry or modify entry 3144?