kloggy / HA-Irrigation-Version2

230 stars 55 forks source link

Time Calculation error? #49

Closed tottelitott closed 3 years ago

tottelitott commented 3 years ago

sd

Only the first Time is used for calculation. Also in Watering Time Overview...

davewatson91 commented 3 years ago

Just checked mine, and is definitely working ok for me.

Have you updated to using the latest version from here?

ahmaddxb commented 3 years ago

I have updated and it not calculating at all. 4 zone 10 min each

image

ahmaddxb commented 3 years ago

Also if i use the option adjust for temperature the watering will only do a minute regardless if it set to 10 mins

davewatson91 commented 3 years ago

Are you adjusting using the slider, or by clicking the zone number, and typing in?

ahmaddxb commented 3 years ago

Either way make no difference to time calculation.

simongie commented 3 years ago

I've had the same error since the beginning too. I've never been able to resolve it.

image

ahmaddxb commented 3 years ago

Mine works correctly now, the reason I had a time calculation error before is because I added more zones and added a duration to those zones and then when i removed those zones time calculation was still taking then into account.

e.g I have 4 zones, added a 5th zone put a duration and then later went back to 4 zones. You need to remove any duration on a zones before reducing the amount of zones in you system.

simongie commented 3 years ago

Interesting. I've never added any additional zones though. It's straight from the repo.

Best regards Simon Gie Mob: +61 (0) 408 161 818

On Wed, 23 Jun 2021, 14:00 ahmaddxb, @.***> wrote:

Mine works correctly now, the reason I had a time calculation error before is because I added more zones and added a duration to those zones and then when i removed those zones time calculation was still taking then into account.

e.g I have 4 zones, added a 5th zone put a duration and then later went back to 4 zones. You need to remove any duration on a zones before reducing the amount of zones in you system.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/kloggy/HA-Irrigation-Version2/issues/49#issuecomment-866601076, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOJ5HIXPEWDXXRP7EPHOFADTUGEQ7ANCNFSM4325ZDIA .

ahmaddxb commented 3 years ago

If you set your zones with eveyday checkbox does the calculation work

simongie commented 3 years ago

[image: image.png] Unfortunately no.

On Wed, 23 Jun 2021 at 15:39, ahmaddxb @.***> wrote:

If you set your zones with eveyday checkbox does the calculation work

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/kloggy/HA-Irrigation-Version2/issues/49#issuecomment-866606668, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOJ5HIUYNNYLEI32QS7F4WLTUGFT5ANCNFSM4325ZDIA .

ahmaddxb commented 3 years ago

Have you tried pulling the files from the repo again and overwritting what you currently have. When is the last time you installed or update HA-Irrigation-Version2 in this way?

simongie commented 3 years ago

I just tried it today, funnily enough! I deleted the lovelace and package directory, pulled down the latest version, then unzipped and moved those folders to their respective positions and restarted HA. Same fault.... it's very strange.

On Wed, 23 Jun 2021 at 15:47, ahmaddxb @.***> wrote:

Have you tried pulling the files from the repo again and overwritting what you currently have. When is the last time you installed or update HA-Irrigation-Version2 in this way?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/kloggy/HA-Irrigation-Version2/issues/49#issuecomment-866611646, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOJ5HIQANHDBYOUIPIZ27D3TUGGSFANCNFSM4325ZDIA .

ahmaddxb commented 3 years ago

hmm if I can think of anything else I'll let you know. I can see i posted earlier a picture that looks like yours but don't remember what i did to make it work.

simongie commented 3 years ago

Just for curiosity's sake, do you have any errors in your log that related to group.set service? This also has something to do with MGIv2 but Kloggy doesnt seem to have the same issue, even though we have similar HA versions.

image

If you don't, do you have anything in your groups.yaml file? Mine is empty. I'm just wondering if we have to populate it with something in order for the service to start. No idea man, I'm willing to try anything at this stage.

ahmaddxb commented 3 years ago

I dont have group.set error's and my groups.yaml is empty.

can you make an automation with the action group.set is it available as and option?

image

simongie commented 3 years ago

That's the weird thing, no I can't. So, I'm not sure if this is related or we are just going off topic...no idea, but it can't be right either way.

image

ahmaddxb commented 3 years ago

I think it's worth try to resolve it and see if that could fix your issue.

simongie commented 3 years ago

I'm just wondering if something didn't carry from older versions of HA. I'm still relatively new, installed maybe a year ago. How long have you been running yours?

ahmaddxb commented 3 years ago

not much over a year now

simongie commented 3 years ago

Baffling...

simongie commented 3 years ago

I fixed it, check the forum! :)

ahmaddxb commented 3 years ago

nice what was it in the end

simongie commented 3 years ago

Needed a create a group for the service to start.

Best regards Simon Gie Mob: +61 (0) 408 161 818

On Wed, 23 Jun 2021, 18:41 ahmaddxb, @.***> wrote:

nice what was it in the end

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/kloggy/HA-Irrigation-Version2/issues/49#issuecomment-866782050, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOJ5HIXAFVK3YQKF2CDR7NLTUHFOPANCNFSM4325ZDIA .