Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Battery to EV Charger #924

Closed
matomatusov opened this issue Jun 13, 2024 · 23 comments
Closed

[Bug]: Battery to EV Charger #924

matomatusov opened this issue Jun 13, 2024 · 23 comments
Labels
bug Something isn't working solax

Comments

@matomatusov
Copy link

Describe the bug

Hi @PatrikTrestik ,
the problem is that when I want to set the EV Charger to Enabled, it switches to Disabled.
1

Integration Version

2024.05.7

Homeassistant core version

2024.6.2

Inverter brand

Solax Power

Plugin used

plugin_solax.py

Serial prefix

H34A10

Inverter firmware versions

ARM1.40 DSP1.41

Connection Method

Packet Wifi + LAN 3.0

Dongle firmware

1.002.09

Detailed Error Log

No response

Additional context

No response

@matomatusov matomatusov added bug Something isn't working solax labels Jun 13, 2024
@PatrikTrestik
Copy link
Contributor

Working on it.

@Henry1-github
Copy link

I have the same problem. :-(

@PatrikTrestik
Copy link
Contributor

Please check what is status of this setting in solax claud app, when you experience this bug in HA.

Does it change or not at all?

@matomatusov
Copy link
Author

If I change it, this message pops up.

Nepodařilo se provést akci select/select_option. Modbus Error: [Input/Output] ERROR: No response received after 3 retries

Sorry for the late response.

@PatrikTrestik
Copy link
Contributor

What does SolaX claud app? Will it change?
Can you set it through app?

@matomatusov
Copy link
Author

yes, it can be set through the application, and when I change it in HA, it is the same in the application. It doesn't change.

@Henry1-github
Copy link

Yes. It is the same by me.

@matomatusov
Copy link
Author

@PatrikTrestik
So I wondered if there would be a problem in the Modbus Register(?).
This item has been moved under modbus in the Solax application.
It was definitely not there.

@PatrikTrestik
Copy link
Contributor

@PatrikTrestik So I wondered if there would be a problem in the Modbus Register(?). This item has been moved under modbus in the Solax application. It was definitely not there.

I think the same.
Next week, when I'm home, I'll go through it.
I have latest (i hope) modbus solax doc.

@Lake292
Copy link

Lake292 commented Sep 1, 2024

Hi, I came across this bug as well today. One more observation, is that the value shown in drop down is actually opposite to what is set in the Solax Cloud. Not sure whether that happens to you as well.

enable
disable

I can confirm as well that setting the value triggers the error in modbus mentioned already and nothing changes in Solax Cloud. My inverter is G4 10kw running 1.41/1.40.

Thank you for looking into this!

@Henry1-github
Copy link

Yes. It is exactly what you write. The opposite in SC and HA. 1.40/1.39

@PatrikTrestik
Copy link
Contributor

There is something happening on SolaX side. I have older firmware (DSP v1.33 ARM v1.31) and this setting completely disapeared from SolaX cloud settings for me.

I'm communicating with SolaX. I ask for firmware upgrade and also for latest Modbus document.
It takes some time.

@matomatusov
Copy link
Author

That setting was moved under the modbus section.

@PatrikTrestik
Copy link
Contributor

I have solution, going to code.

TLDR: Write register (0x0E1) is different than read register (0x10F). Happy SolaXing 😄

PatrikTrestik added a commit to PatrikTrestik/homeassistant-solax-modbus that referenced this issue Sep 2, 2024
@PatrikTrestik
Copy link
Contributor

I hope that it is fixed in #1033

My SolaX app does not show the option so I can only test HA entity. Please check if it is aligned with App.

@Henry1-github
Copy link

Nothing has changed in my case. :-(

@PatrikTrestik
Copy link
Contributor

Nothing has changed in my case. :-(

You installed from PR mamually?

@Henry1-github
Copy link

You installed from PR mamually?

What do you mean by PR?

@PatrikTrestik
Copy link
Contributor

You installed from PR mamually?

What do you mean by PR?

This is Github. It works on principles of Git, which uses Pull Requests for code changes.
For more info see help pages.

In my comment you see that it is fixed in PR #1033.
You have to manually get that code and install it if you want to test not yet published code.

This repo owner has to prepare new release version which will contain this change.
If you are not developer, you have to wait.

@Lake292
Copy link

Lake292 commented Sep 2, 2024

I can confirm that this fixes it. Reading and writing is properly reflected in Solax Cloud.
Thanks again for looking into this... simple fixes are best :)

@PatrikTrestik
Copy link
Contributor

@wills106 please merge PR. It was tested. @Lake292 thank you for test.

wills106 added a commit that referenced this issue Sep 5, 2024
…rger

#924 Fixed value mapping "battery_to_ev_charger"
@wills106
Copy link
Owner

wills106 commented Sep 6, 2024

I have released 2024.09.1 with this fix.

@matomatusov
Copy link
Author

Hi,
thanks for fixing the problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working solax
Projects
None yet
Development

No branches or pull requests

5 participants