Pre-release 24.07.29 causes issues when are not scheduled requests in every coordinator update loop

davidrapan / ha-solarman

⚡ Solarman Stick Logger integration for 🏠 Home Assistant

MIT License

110 stars 25 forks source link

Pre-release 24.07.29 causes issues when are not scheduled requests in every coordinator update loop #49

Closed gedger closed 3 months ago

gedger commented 3 months ago

Just tried the above and it raises a load of errors. Downgrading to .28 fixes.

Sorry, I've lost the log....

No I haven't, looking at it now it appears to have recovered just as I moved back to the previous version....I'll try again when I have some time just in case the inverter really was off-line?

home-assistant_solarman_2024-07-30T08-21-37.172Z.log

davidrapan commented 3 months ago

As it's unfortunate this pre-release version is mainly for testing some niche cases of some users which are not able to run it at all i will proly change multiple times before the release. But still thanks for the report for sure! I'm also running it but without any problems, maybe it's device dependent hm?

gedger commented 3 months ago

For info I've just tried the latest from main and it still causes issues so will need a tweak before releasing. Log attached which will hopefully help.

home-assistant_solarman_2024-07-31T14-46-38.733Z.log

davidrapan commented 3 months ago

~~Not sure what exactly could cause that but I tried: fix: Reverted partial request fulfillment~~ as it turned out this feature is not necessary and original issue was caused by incorrect profile used.

I'm taking that back. I know exactly what is happening.

Edit: fix: Reverted partial request fulfillment

Edit2: Can I ask you for something? Can you show me detail view of the Update Interval sensor? So is it's history for last day is visible? (Best time for capture would by around the time when your device goes to sleep)

gedger commented 3 months ago

Sorry, only just seen your Edit2: above. I have been seeing some really variable update intervals which I think has been also caused by the auto discovery. I have now disabled auto discovery which has improved matters plus have loaded 24.07.31 with no issues so far.

Going off-line

Coming on-line

I'm still struggling to debug what happened at 8.05am when I restarted HA with a slightly different profile that had 3 disabled parameters. Now I've disabled auto discovery and running the latest build I will evaluate again over the coming days.

davidrapan commented 3 months ago

Could you maybe do a test for me with 5 sec update interval for lets say a hour during the day? And also share the graph here?

gedger commented 3 months ago

Just realised, would you like a debug log too? If so I'll run again.

davidrapan commented 3 months ago

Thanks! And no it is not necessary I just wanted to see what are the usual update intervals on other installations.

Very appreciated!

davidrapan commented 3 months ago

What about the issues you were seeing? Are they still present in the last pre-release?

gedger commented 3 months ago

Latest release has been fine, I was just giving it a few days to confirm. I couldn't determine the cause of the variable polling time when it varies between 30-40s as shown on the "coming on line" graph above. Integration, inverter or network? If I ever find a way of provoking it then I'll raise an issue (if it is the integration) so I think it's safe to close this.