magtimmermans / com.cyclone-software.sunevents

First Beta Release
6 stars 9 forks source link

Sunset, sunset start flow not work for me #45

Closed felho001 closed 1 year ago

felho001 commented 3 years ago

Hi when I create a flow e.g Sun Events (offset in min) Sunset 20 min an the time is after sunset and before sunset 20min. is work and run and I see the app on trigger like this:

Képernyőfotó 2021-05-02 - 20 31 13

But if I want to other flow and it is after sunset all wariable I want the app not see the trigger and not work tomorow. like this:

Képernyőfotó 2021-05-02 - 20 32 37

And if I create one e.g the other flow before sunset time like this Sun Events (offset in min) Sunset 20 min the trigger on the app but only real sunset time show not sunset time +15min.

Is it a bug or I was lame?

magtimmermans commented 3 years ago

Hi,

I hope I understand you correctly. But you see only the active triggers from that day and not tomorrow. If the time is passed they are gone.

felho001 commented 3 years ago

Hi,

I hope I understand you correctly. But you see only the active triggers from that day and not tomorrow. If the time is passed they are gone.

OK thanks I see.

But look my 2 flows. and real sunset time the app.

Képernyőfotó 2021-05-03 - 6 36 25

Here my flows:

Képernyőfotó 2021-05-03 - 6 37 28 Képernyőfotó 2021-05-03 - 6 37 11

and time the trigger in app:

Képernyőfotó 2021-05-03 - 6 37 55

and the time is same Sun Event app time, sunset and sunset start. but flow is not started after sunset 15 min. and after sunset start 15 min. The set extra time (15min) is not added to the real time on trigger and not strated flow.

Képernyőfotó 2021-05-03 - 6 55 02

I create one flow and the time when I created e.g 19:56 after 1 minute sunset start the trigger time in app I will see 20:15 sunset start and flow run correct.

magtimmermans commented 3 years ago

Hi,

Which country are you in? Do you have the coordinates set correctly? Please make a simple flow and write it to the timeline like: image

Wait at least one day to check to be 100% sure.

felho001 commented 3 years ago

HI I leave in Hungary and the homey geofacing see correctly. I will test like you send. Here is a test flow. https://homey.app/f/OrHvNA It's time sun event 12:41. And the flow time started 12:51.

felho001 commented 3 years ago

Hi I tested flow but nothing happened only app update!image

felho001 commented 3 years ago

This is I told you. I create new solar noon flow after the solar noon start, and now you se the solor noon flow trigger time is correct and I sure this flow run correct. But see other trigger in de list is it sunset time not a flow time because the flow is after 10 min start when sunset Time. image

magtimmermans commented 3 years ago

image

image

image

It should work or we misunderstanding each other. And are you sure that you do not have more flies running? If you mean that you see 2 items

felho001 commented 3 years ago

Ok i TRY show you.. the first is solar noon the times is 12:41 when start my location I set one flow after real time 12:41 It was 12:55. And you see my picture the flow will run 13:41 because I set 60 minutes. And other one I set sunset time after 10 minutes. This set I make now. And you see the picture the trigger not change same time the sunset time my location and not more then 10 minutes. And same problem I set solar noon before 12:41 and flow not started. A830E5A3-5DB6-4207-BD60-171484E8176F 2C7FD2A8-C464-4FDC-8C31-62D5F0D54982 1DFF7024-A613-4AC7-9410-0615BAAD893A

felho001 commented 3 years ago

D47382D6-6C90-4494-8DCB-9DA40B465783 Ok I now what’s the problem! If I set the times logic variable not update the time 10 min 15 min or other times, only see the trigger normal times e.g sunset is today 19:58. But if I only write number the times will correct end I sure the flow is run perfect. Here some logic in picture and the 2 different 1 flow is logic set, and same flow is only number sunset trigger. 18302097-2583-49C0-AE4C-314918816B2F B510CA1C-CAC7-473F-ABBF-85758B7AAE12

magtimmermans commented 3 years ago

Ok, So it is working ?

felho001 commented 3 years ago

yes, but the logic variable not working! could you fix this?

felho001 commented 3 years ago

I create new 1 with logic. This is wrong!!

Képernyőfotó 2021-05-03 - 21 16 45 Képernyőfotó 2021-05-03 - 21 17 21

this is a flow:

Képernyőfotó 2021-05-03 - 21 20 32
felho001 commented 3 years ago

and I create one only number

Képernyőfotó 2021-05-03 - 21 16 45 Képernyőfotó 2021-05-03 - 21 22 15

end it is a only number flow without logic this look good.

Képernyőfotó 2021-05-03 - 21 22 26

But if you can fix this I love it.:-)

magtimmermans commented 3 years ago

I think I finally understand you. When you create a trigger instead of filling in a number you use a tag.

Like from this list: image

That's what you mean? Can you tell me which tag from which app you use? From homey itself?

felho001 commented 3 years ago

Here is my tag logic and I use the tag e.g min where neeed number and minutes and I e.g. use evohome heating app

Képernyőfotó 2021-05-03 - 22 39 22
felho001 commented 3 years ago

IMG_0048 IMG_0047 IMG_0046

felho001 commented 3 years ago

Yes thats the problem if I use tag. Hue, Evohome, sensibo, frient apps use tag

felho001 commented 3 years ago

I changed the tag name e.g. 15 min changed fifteen minutes. And now trying this new tag without number of names. I think this is the problem issue.

felho001 commented 3 years ago

No this idie not solved the problem. tag not work

magtimmermans commented 3 years ago

OK, clear. I will try to fix this in the weekend.

felho001 commented 3 years ago

Thanks!!

magtimmermans @.***> ezt írta (időpont: 2021. máj. 4., K, 8:44):

OK, clear. I will try to fix this in the weekend.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/magtimmermans/com.cyclone-software.sunevents/issues/45#issuecomment-831717736, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKYG6SPGBCS4HCEN6QJUYP3TL6JWDANCNFSM437VV5WA .

-- Üdvözlettel: Dávid Zsolt +36304004009

magtimmermans commented 3 years ago

Hi, I have look into this and it is a bug from homey in sdk 2. I will report this tomorrow as I have discussed this with a developer from athom. Instead of getting the value back I get the object. Thus is not correct. So I have to wait for a fix in homey.

felho001 commented 3 years ago

thanks for information. This may be true for all applications that use this number tag?

magtimmermans commented 3 years ago

Yes, if it is een sdk 2 app and the latest version of homey.

felho001 commented 3 years ago

Yes, if it is een sdk 2 app and the latest version of homey.

Ok but now have SDK3. And If you update the sun events new SDK? Is it possible or not?

magtimmermans commented 3 years ago

That's not very easy and costs a lot of time.

magtimmermans commented 1 year ago

It is now SDK3 but this error remains there as Athom did not fix their code. https://homey.app/a/com.cyclone-software.sunevents/test/