magtimmermans / com.cyclone-software.sunevents

First Beta Release
6 stars 9 forks source link

Azimut values incorrect #56

Closed Joost2 closed 2 years ago

Joost2 commented 2 years ago

Hi, thanks for this great App. When I check (May 9th) the azimut value on Solar events and then on sunearthtools.com, there a difference of 15, one hour (Solar events is too late). Where does the App get his data from, is there a way to correct this (potentially the position on the globe is wrong)? Thanks for your help!

magtimmermans commented 2 years ago

I am using the same calculation as in https://www.suncalc.org/

Joost2 commented 2 years ago

There's a one-hour difference between https://www.sunearthtools.com and https://www.suncalc.org/. According to my solar-panel production, the sun culminates around 12:45, which is one hour earlier than what is indicated on the suncalc.org site and corresponds to sunearthtools.

dirkg173 commented 2 years ago

I use the SunSurveyor App and it shows the same values like the Homey Sunevents App. Maybe there is a difference because the UTC time isn't correct on https://www.sunearthtools.com.

Another possible point, with Homey firmware v7.4.x Node.js v16 is used. And with Node.js v16 the timecode changed as I read in the forum and on Slack. @magtimmermans, I'm not a programmer but could this be a possible reason?

magtimmermans commented 2 years ago

I think indeed that https://www.sunearthtools.com/ incorrect because of the wrong time zone . If I compare this with home assistant (default build in) then they are the same as well. Also several other tools have the same value. If I check when my backyard gets sun it is correct. I could be wrong but I think that https://www.sunearthtools.com/ use gmt+0 I instead of CET.

Joost2 commented 2 years ago

I finished up using “event nadir with offset of x minutes", x being calculated separately and stored as a variable. However, it didn’t work. The “filtration nadir…” variable was 401, but my pool pump didn’t turn on around 9, as would be expected. I re-tested with a (numerical) variable of which I manually determined the value and again no action. Why’s that ?

Thanks !

Joost

On 9 May 2022, at 10:55, magtimmermans @.***> wrote:

I am using the same calculation as in https://www.suncalc.org/ https://www.suncalc.org/ — Reply to this email directly, view it on GitHub https://github.com/magtimmermans/com.cyclone-software.sunevents/issues/56#issuecomment-1120829375, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZCT7W34M4Q4V6TVVQEA7HTVJDHIXANCNFSM5VNPV6UQ. You are receiving this because you authored the thread.

magtimmermans commented 2 years ago

That's a known issue in Homey. That variables are not fleeting up correctly. Infact this issue (if I understands you correctly) is already open for more then a year :( very frustrating. It seems difficult to fix.

I think you can better store both into a var and have an automation that checks when the event is changed and conditions are met and then execute what you want. Kind of complex but that the way around the bug.

Joost2 commented 2 years ago

Needs to be fixed though, where "can I complain” about this handicapping issue ? Thanks for, as always, your rapid answer, very helpful (even if it did not solve my problem). Joost

On 10 May 2022, at 18:23, magtimmermans @.***> wrote:

That's a known issue in Homey. That variables are not fleeting up correctly. Infact this issue (if I understands you correctly) is already open for more then a year :( very frustrating. It seems difficult to fix.

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

magtimmermans commented 2 years ago

https://github.com/athombv/homey-apps-sdk-issues/issues/204

Joost2 commented 2 years ago

I tried with a fixed number:

Why the “number” field is red instead of blue ? It will accept the number, but still… Is there a limit? The number indicates minutes, right?

On 10 May 2022, at 20:13, magtimmermans @.***> wrote:

athombv/homey-apps-sdk-issues#204 https://github.com/athombv/homey-apps-sdk-issues/issues/204 — Reply to this email directly, view it on GitHub https://github.com/magtimmermans/com.cyclone-software.sunevents/issues/56#issuecomment-1122715439, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZCT7WZAZZ7JYROLYMQWKVTVJKRK7ANCNFSM5VNPV6UQ. You are receiving this because you authored the thread.

magtimmermans commented 2 years ago

Hi, yes limit is 60 minutes

Joost2 commented 2 years ago

You can’t make that 24 hours? It should really help. Well, directly entering 470 minutes actually works fine, but doing that via a variable doesn’t, could that be the issue?

On 10 May 2022, at 22:13, magtimmermans @.***> wrote:

Hi, yes limit is 60 minutes

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

Joost2 commented 2 years ago

In any case with a variable with a fixed value of 50 it still doesn't work but it won’t harm to increase the limit, in case a solution will be found. Thanks again !

You can’t make that 24 hours? It should really help. Well, directly entering 470 minutes actually works fine, but doing that via a variable doesn’t, could that be the issue?

On 10 May 2022, at 22:13, magtimmermans @. @.>> wrote:

Hi, yes limit is 60 minutes

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

magtimmermans commented 2 years ago

Okay, when I have some time I will check if this is possible to update. The var issue I am depending on homey dev team.

magtimmermans commented 2 years ago

I cannot make it 24 hours as in interfering with the next event.