-
Notifications
You must be signed in to change notification settings - Fork 23
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
Add native support for Sunrise/Sunset #93
Comments
@NickWaterton I've done some refactoring of the scheduler. Would you be willing to test the current dev branch? There is a new function to run on sunrise, sunset, etc. |
Ok, i just pulled the dev branch, it says it's V0.10.1 which what I already have (but I'm on the dev branch now). Everything started up OK. No errors. What would you like me to test? |
I forgot to update the version number on the dev branch - sorry. |
OK, everything is working right now. Just FYI, I have just completed conversion of 370 OH rules to HABApp! Only a few left (difficult ones though). |
Oh wow - you sure have lots of them! |
Most of them are one-liners, as OH forces you to have a rule for everything... So I have been testing sunrise and sunset events. No Sunset event yet (even at sunset), but I did get a sunrise event:
Notice it's logged at |
Have you set the correct location in the habapp.yml? |
Probably not, I was thinking it would use the environment variables. I'll go check it now. Ok where is OK, ignore me, I assume it's lat/lon in |
Sunset event just occurred right on schedule... |
Sorry, sometimes I don't remember the right filenames 😦 |
- Added _last_change and _last_update to item and created property which returns timezone naive obj - Item value get properly set if they return a unit - Added support for ThingStatus events - Added fixes and tests for QuantityType - Added mapping of QuantityType (fixes #94) - Scheduler fixes - Added scheduler for Sunrise/Sunset/etc (closes #93) - Internal Timestamps are all utc now - Added location config
Calculation can be done through the Astral package
The text was updated successfully, but these errors were encountered: