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

Initial fixes and skeleton for Solax PM mode support #1076

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fxstein
Copy link
Contributor

@fxstein fxstein commented Oct 3, 2024

This is work in progress not ready for merge. I opted to open a pull request early - even thought the changes are not final, to get more eyes on the parallel mode problems.

This patch so far does the basics:

Identify when inverter is part of a parallel mode setup with additional inverters present. Modifies various calculations to account for changes in parallel mode behavior.

If you have a parallel mode setup for testing or simply have thoughts, please comment away. I have the ability to test against a 3 way parallel system.

Identify when inverter is part of a parallel mode setup with additional inverters present. Modifies various calculations to account for changes in parallel mode behavior.
@fxstein
Copy link
Contributor Author

fxstein commented Oct 4, 2024

@wills106 Here is a question I have:

I am trying to
understand the logic and source of the data for the I2 and I3 sensors, I can see the modbus registers but have not found those in the modbus documentation I have. Where can I find more details about these?

Thank you!

@fxstein
Copy link
Contributor Author

fxstein commented Oct 5, 2024

@wills106 Here is a question I have:

I am trying to understand the logic and source of the data for the I2 and I3 sensors, I can see the modbus registers but have not found those in the modbus documentation I have. Where can I find more details about these?

Thank you!

I found a more up to date modbus documentation that has the parallel mode slave devices.

@romankysely
Copy link

Fine, could you share this resources? thx

@fxstein
Copy link
Contributor Author

fxstein commented Oct 5, 2024

Fine, could you share this resources? thx

Wont be the latest, but covers the PM devices list: https://gbc-solino.cz/wp-content/uploads/2022/07/Hybrid-X1X3-G4-ModbusTCPRTU-V3.21-English_0622-public-version.pdf

@romankysely
Copy link

Thx. There is newer a little bit. v3.29
(Solax)Hybrid X1&X3-G4 ModbusTCP&RTU V3.29-English_231031 .pdf

@fxstein
Copy link
Contributor Author

fxstein commented Oct 6, 2024

Ok made quite a bit of progress the past few days. During extensive testing I finally found a measured power logic bug in the core Solax logic. I will be opening a separate PR for that shortly.

@fxstein
Copy link
Contributor Author

fxstein commented Oct 6, 2024

Opened a separate PR for the meter direction bug: #1082

@fxstein fxstein marked this pull request as draft October 6, 2024 21:22
@Pukkahq
Copy link

Pukkahq commented Oct 19, 2024

I have 2 Solax SK-SU5000E Modbus/TCP Interface (Solax-Modbus) inverters and have been running them in a pair via this code for a few years. I am happy to dive in to help and test, will your plug work roughly the same?
I am getting the import/export via the CT clamp on solax1 inverter.

@Pukkahq
Copy link

Pukkahq commented Oct 19, 2024

I thought I would install it and give it a go. I am not sure if I have the config correct, but you can see the problem having 2 inverters. 1 is charging while the other is discharging.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants