-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
All icons are night icons #79
Comments
Thanks for bringing this up- I hadn't noticed it before, but I can verify it's still an issue for this region. The icons are working correctly in North America; however, I can also see that the sunrise/ sunset times are not working, so I think that's the root problem here. I'll dig into this and get back with a fix ASAP! |
Ok, got it, another literal edge (of the map) case! The issue here was with the error handling when there isn't a sunset, since the algorithm defaulted to assuming it was always night, instead of working out if it should be always day or night. I fixed this by checking the month and location, and then picking "always day" or "always night" from there. The API will then return a sunrise of 1 second after midnight and 1 second before midnight in the case of "always day", or a sunrise of 2 seconds before midnight and a sunset 1 second before midnight for "always night". I'll make version 1.5.3 with this change live shortly, and update the changelog/ docs with this corrected approach. Thanks again for pointing this out, and let me know if you run into any other issues. |
There has not been any activity on this issue in the last ninety and will automatically close in seven days. Comment on this issue to prevent this issue from closing automatically. |
This issue has been automatically closed since there has been no further activity after seven days. |
It seems all icons which can have a "day" or "night" suffix use "night" despite the sun being up at this location. The response to
https://api.pirateweather.net/forecast/.../61.4981,23.7604?exclude=minutely,hourly,daily,alerts
at 14:56 local time:The text was updated successfully, but these errors were encountered: