-
-
Notifications
You must be signed in to change notification settings - Fork 102
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
[FR] Support solarmanv5 protocol #264
Comments
I would look / ask in here #188 |
I think the port number was the first thing I checked. Solarman requires providing the stick S/N. Mine is 10 digits starting with I will also check the plugin |
You are probably best checking on that post how others are connecting. I thought the WiFi stick didn't provide access. But they are using something other than a RS485 adaptor by the looks. |
@wills106 My WiFi stick works with Solismon and Solismod, which is based on Modbus communication. Any ideas on how we could have this supported? |
Hi @wills106 this wrapper is based on pysolarmanv5 and it introduces reading/writing to Modbus registers of the Solarman-based WiFi sticks. I also wrote a script to read and write registers - it's based on examples from Solismon3 |
I'm not sure why your Inverter isn't connecting if other people are using the WiFi stick. Unless there are different version of the stick? The serial 6031 has been tested by other users. |
@wills106 I got it... the WiFi stick I have needs the Modbus RTU (pysolarmanv5) to communicate as it doesn't support native Modbus TCP like some LAN modules. The main question is if there is an easy way to integrate pysolarmanv5 into your integration. |
I can confirm that the DLS-L data logger on my Solis Hybrid Inverter works perfectly with this integration. It could also work with DLS-W (Wi-Fi data logger) if this integration supports the pysolarmanv5 protocol. @wills106 any ideas on how this could be achieved? |
I have had a quick look at the protocol and I'm not sure if I have time at the moment to support it. I don't think the current multi register read code would work as it looks like they build multiple reads differently and I am not sure if two register read is automatically turned into a 32bit register. So we would have to be careful not to have the auto builder read a block of two registers. We would also have to modify the number, selects & sensor files etc to use the alternative method and either rework a plugin file just for solarmanv5 or translate it with perhaps a checkbox on initial setup to tell the Integration to use an alternative method to Modbus. |
Thanks @wills106 I thought it would be the case. For now, the DLS-L should be enough for me now, but expanding the support to DLS-W will make this integration available to many others. |
Describe the bug
Cannot make it working for some reason.
Mandatory details
Detailed Error Log
Additional context
I am wondering how to fix this. It works with other integrations e.g., https://github.com/StephanJoubert/home_assistant_solarman but has many issues with data, so I am looking for alternatives.
Any advice?
The text was updated successfully, but these errors were encountered: