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

Synchronise the MODBUS commands with the inverters's state #111

Closed
s-allius opened this issue Jun 23, 2024 · 1 comment
Closed

Synchronise the MODBUS commands with the inverters's state #111

s-allius opened this issue Jun 23, 2024 · 1 comment
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@s-allius
Copy link
Owner

In the proxy version 0.8 we send MODBUS commands regular by a cron job. Especially the GEN3 inverters are picky when they receive unexpected frames. Tests shows, that the inverter firmware seems to be much robuster when we send MODBUS command not in the phase when the inverter reports the data.

We want a timer in the protocol state machine, which triggers the MODBUS commands.

GEN3 inverter

For GEN3 inverters the timer should be startet/restarted when the inverter sends a new get time request. This indicates the beginning of a new reporting phase. Since the phase is about 1 minute long a timeout of 90sec should garantie that no MODBUS commands will be send during this phase.
After sending a MODBUS command the timer should be startet with a timeout of 60sec. So the cadence of the regular MODBUS request will be about 1 minute

GEN3PLUS inverter

These inverters are very robust and there are no restrictions to consider.
We can start the timer as soon as the first packet is received from the inverter and then simply start it again when sending MODBUS commands. A fixed timeout of 60sec will do.

@s-allius s-allius added bug Something isn't working enhancement New feature or request labels Jun 23, 2024
@s-allius s-allius self-assigned this Jun 23, 2024
@s-allius
Copy link
Owner Author

fixed by version 0.9 #112

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

No branches or pull requests

1 participant