-
Notifications
You must be signed in to change notification settings - Fork 16
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
Cooperative Mode for S-Miles cloud with parameter to enable / disable it. Will pause polling the inverter at appropriate times #110
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Dennis Luxen <ich@dennisluxen.de>
Hi @DennisOSRM, Could you check if the optional option works this way? Sorry, it is my first contact with Rust and it looks a bit weird to me ;) |
Hi again, |
Cool, thanks for the info! In terms of timeline I plan to put the branch on my test setup on Thursday and if nothing else comes up to merge over the weekend. |
Co-authored-by: Dennis Luxen <ich@dennisluxen.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking good, thanks so much for the contribution
I'm pretty sure that this 'skip 14th minute' idea is not the best solution cause the cloud updates happen a lot more often, the web site just accumulates it every 15 minute.
My conclusions are:
I will now try update_interval: 45000 which should make the phases of failure shorter than 4-5 minutes. |
TCP sessions of cloud update a little more stable but missed 3 of 4 updates. Not what I expected. |
@Matthies have you tried logging Cloud-Updates when HA-Update isn't running? Just curious if the connection is stable in that case. |
Will do this weekend. |
@m4rquez I have stopped HA-update now for 30 minutes and captured communication between HMS-800W-2T and Hoymiles Cloud. As expected the cloud updates can be seen every 60 seconds, this time rock stable all within the same TCP session. |
One idea that crossed my mind recently was to reimplement sending packets into the cloud from the tool itself. That has to be explored more deeply though |
Hmm. Not sure if that is possible. If I would offer cloud services for inverters, I would secure communication with some certificate protected by some private key stored inside the inverter/DTU. |
Yes, sure. Yet that particular cloud service doesn't have much of that. 😉 |
thanks @Matthies , even with updates every 60sec, i'm missing updates in the cloud sometimes. sending packets from the tool to the cloud sounds cool, but i'm sad, such a workaround seems to be neccessary. when you are connected to the inverters wifi directly, it is possible to get updates of the current power in the app, i guess, every second. maybe this is another endpoint? possibly this can be used to get regular updates of the current power without disturbing cloud communication? |
Introduces a flag that disables polling the inverter at the 14th minute of each hour.
Likely fixes #52 more reliably than setting a higher update interval.
cargo clippy
and fix all issues (command didn't run for me, sorry)cargo fmt
to format all source filesPS: I am no Rust dev + I have not included the new parameter "smiles_cooperation" anywhere for the docker config, so maybe that still needs to be done.