-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sunevents empty #48
Comments
@Giel88, please can you provide more details?
|
Sure! I am running the homeassistant addon
|
I have just updated the node-red on my development system from v3.1.3 to v3.1.9, testing sunevents before and after with your same coordinates, and the results were the same both times. I don't see an issue. I am looking at your screen shot. You say that the sunevents array is empty, however your screenshot clearly shows there is one element for "night" in that array. Please clarify. |
Aren't both outputs supposed to give me a timestamp for either upcoming or past sunevents? The night timestamp is empty and all other events are missing |
OK, now I understand the problem. You meant the timestamp was null not the that the array is empty. Big difference. |
Correct, mistake on my end. Sorry about that! |
Since an unknown point in time, sunevents has also been behaving completely incorrectly for me, contrary to the documentation, and is currently not providing any usable data. I didn't knowingly make any updates, but I can't assess whether automatic update processes might have taken effect. |
I'm sorry this node has stopped working. I can only assume that a change in some dependency has this problem. Unfortunately I won't be able to look into this for a while. |
That's alright, life gets busy sometimes! @IronSolder For now I'm using the sunrisesunset.io API, maybe that's a solution for you as well. |
External dependencies are a plague... . |
The past sunevents array is empty (null) when injecting. Worked before but no longer works after nodered update! Nodered log gives no errors only:
21 May 19:18:26 - [info] [sun events:c1ee1b226fd00ee2] calculating sunevents for lat 52.10347 and lng 5.10608
The text was updated successfully, but these errors were encountered: