jeroenterheerdt / HAsmartirrigation

Smart Irrigation custom component for Home Assistant
MIT License
301 stars 47 forks source link

Doubled zone duration caltucaltion #356

Closed LPX64 closed 1 day ago

LPX64 commented 2 weeks ago

What happened?

Dear Jeroen,

First of all I would like to thank you a lot for creating such smart and helpful integration. I cannot imagine my garden without it. 😊

I would like to report an issue with zones calculation. At the beginning of my adventure with smart irrigation I have set the time of zones calculation at 4:45. Then, after several days I have changed it to 23:45, but to my surprise zone are being calculated twice. At 23:45 and 4:45. It looks like parameter ‘Automatic duration calculation’ can store many values but only latest one is displayed. Please find attached files. A chart showing double zones calculation, system settings print screen, and system logs (the only ones from 4:45 that I can find). There are no other logs except one with status error, but it seems that it was connected with access to open weather map.

ustawienia wykres

How to reproduce

Change parameter 'Calculate at' at general settings, under 'automatic duration calculation'.

Relevant log output

Ten błąd pochodzi z niestandardowej integracji.

Rejestrator: custom_components.smart_irrigation
Źródło: custom_components/smart_irrigation/__init__.py:693
integracja: Smart Irrigation
Pierwsze zdarzenie: 21 kwietnia 2024 04:45:02 (28 zdarzenia)
Ostatnio zalogowany: 04:45:02

Calculate for zone Linia kroplujÄ…ca failed: no data available.
Calculate for zone Trawnik A Trigger failed: no data available.
Calculate for zone Trawnik B Trigger failed: no data available.
Calculate for zone Trawnik C Trigger failed: no data available.

Which version are you running?

v2024.4.3

Diagnostics file

Additional information

config_entry-smart_irrigation-60df4432ab5c5a5ae27f69bb3c6b1183.json

LPX64 commented 1 week ago

Issue not longer persist after new HA version installed. I assume that HA restart was enough. But bug is still there and if you change the time of zones calculation - you need to restart HA to remove old schedule.