Program don't start at the time

petergridge / Irrigation-V5

Irrigation custom component for Home Assistant

GNU General Public License v3.0

54 stars 10 forks source link

Program don't start at the time #18

Closed Loic691 closed 11 months ago

Loic691 commented 1 year ago

Hi I'm testing irrigation v5 When I want to check program, I change the start time to 1 or 2 minutes after now and the program don't start. The program starts correctly when I click on RUN.

What is the procedure for the changing time. Do I need to restart HA ?

Thanks for your help

petergridge commented 1 year ago

Hi, I guess you have the frequency set using a numeric e.g. every 2 days. If the program has run successfully and you change the time it will still be calculating the number of hours until that frequency allows it to run again. i.e. it sill only completed 2 hours ago and your setting is to run every 24 hours.

You have two options:

change to days of the week for your testing
use the irrigationprogram.reset_runtime service to reset the last run time

Loic691 commented 1 year ago

Thanks Should I have to launch in manual before test ? Here's an other test fails : Frequency is set to 1 (every day) Time is 21h30. I set program to run at 21h35. I called the reset_runtime. At 21h35 the progam hadn't been launched

I tried too to change the runtime time just after service reset_runtime ( to 1 or 2 minutes after now). Always non program starting...

When is calculated the new starting date ?

When changing time in starting-runtime ?
At the end of the programm runs sucessfully ? PS : the calling of service reset runtime reset correctly the field in the frontend (if i have a value du to before manual launch)

Loic691 commented 1 year ago

Hi again Program working now I forget to ceate the time_date Pre-requisite ;-(

jschroeter commented 11 months ago

I had exactly the same problem and missed the time_date integration. Not sure if it would be possible to display a error message in this case. Anyway, thanks a lot for your great work, seems to work as expected now!

petergridge commented 11 months ago

Hi @jschroeter, I will remove that dependency, I have been meaning to code that in for sometime now but it never got very high on my to do list, it worked even though I knew I could do better. Anyway the change will be in the next release.

petergridge commented 11 months ago

Release 5.2.6 (beta) has removed the dependency on time-date sensor. I would appreciate if you could install the beta and verify the release.

jschroeter commented 11 months ago

Thanks a lot, just tried it, works like expected without the time-date sensor 🥳