Hey @Gary Waterworth,
Thank you for developing your scripts, they are great!
I had some issues however when setting up your scripts, due to various issues, such as:
Using complete docker compose/swarm setup, where it was not an "addon"
Performance issues when trying to check inverters which don't exist in a master/slave setup
Requesting plant list from a complex account of different inverters, setups, masters and slaves - due to a multi-plant account
Therefore, ive fixed many issues, and I've put a PR on Github
Here are the list of changes on the PR - https://github.com/gdwaterworth/Home-Assistant---E-Linter-Logger---Sunsynk/pull/52
Please note: Myself and Gary are currently working through before merging the changes, so they may change!
Multiple lists of plants can have master and slaves and different size inverters, which broke part of the flow -some common errors ive seen other users post here should now be fixed.
Time uploads broke as didn't follow the correct format - mainly due to slave plants trying to send zero
This config has been tested with the latest Docker containers of HA 2024 and NR v3
Added additional debugging for debug mode
Display the debug and warnings state is enabled/disabled within Flow and debug msgs
Performance improvements, which skip Plant ids where an inverter id and data doesn't exist (usually slave plants) - this helps with multi-plant accounts, also where an inactive plant may exist within the list also
Tidy up debug messages,
Added friendly Warnings to help with setup issues for first setup, as current wiki not clear
If anyone would like to try my changes, please backup your current flow first, try at your own risk, etc - the changes are here: https://github.com/t0mtaylor/Home-Assistant---E-Linter-Logger---Sunsynk/tree/Stable-Release
Next changes I may look at, is allowing the script to support multiple plant and inverters being submitted to Home Assistant
Thanks
Tom