{"action":"created","issue":{"url":"https://api.github.com/repos/balloob/home-assistant/issues/991","labels_url":"https://api.github.com/repos/balloob/home-assistant/issues/991/labels{/name}","comments_url":"https://api.github.com/repos/balloob/home-assistant/issues/991/comments","events_url":"https://api.github.com/repos/balloob/home-assistant/issues/991/events","html_url":"https://github.com/balloob/home-assistant/pull/991","id":128426179,"number":991,"title":"Makes time triggers use crontab","user":{"login":"kennedyshead","id":203370,"avatar_url":"https://avatars.githubusercontent.com/u/203370?v=3","gravatar_id":"","url":"https://api.github.com/users/kennedyshead","html_url":"https://github.com/kennedyshead","followers_url":"https://api.github.com/users/kennedyshead/followers","following_url":"https://api.github.com/users/kennedyshead/following{/other_user}","gists_url":"https://api.github.com/users/kennedyshead/gists{/gist_id}","starred_url":"https://api.github.com/users/kennedyshead/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kennedyshead/subscriptions","organizations_url":"https://api.github.com/users/kennedyshead/orgs","repos_url":"https://api.github.com/users/kennedyshead/repos","events_url":"https://api.github.com/users/kennedyshead/events{/privacy}","received_events_url":"https://api.github.com/users/kennedyshead/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2016-01-24T21:24:52Z","updated_at":"2016-01-26T01:38:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/balloob/home-assistant/pulls/991","html_url":"https://github.com/balloob/home-assistant/pull/991","diff_url":"https://github.com/balloob/home-assistant/pull/991.diff","patch_url":"https://github.com/balloob/home-assistant/pull/991.patch"},"body":"Blackbox tested as far as I could come up with tests, if there needs to be some unittest I'll gladly write them I was not sure if this is covered already.\r\nTest by:\r\n```\r\nautomation:\r\n  trigger:\r\n    platform: time\r\n    cron: '*/2 * * * *'\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/balloob/home-assistant/issues/comments/174761445","html_url":"https://github.com/balloob/home-assistant/pull/991#issuecomment-174761445","issue_url":"https://api.github.com/repos/balloob/home-assistant/issues/991","id":174761445,"user":{"login":"rmkraus","id":4956442,"avatar_url":"https://avatars.githubusercontent.com/u/4956442?v=3","gravatar_id":"","url":"https://api.github.com/users/rmkraus","html_url":"https://github.com/rmkraus","followers_url":"https://api.github.com/users/rmkraus/followers","following_url":"https://api.github.com/users/rmkraus/following{/other_user}","gists_url":"https://api.github.com/users/rmkraus/gists{/gist_id}","starred_url":"https://api.github.com/users/rmkraus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rmkraus/subscriptions","organizations_url":"https://api.github.com/users/rmkraus/orgs","repos_url":"https://api.github.com/users/rmkraus/repos","events_url":"https://api.github.com/users/rmkraus/events{/privacy}","received_events_url":"https://api.github.com/users/rmkraus/received_events","type":"User","site_admin":false},"created_at":"2016-01-26T01:38:29Z","updated_at":"2016-01-26T01:38:29Z","body":"I think you should give some thought as to what problems you are trying to solve and make sure that you are only solving those problems.\r\n\r\nIf you want to schedule things for every 5 minutes (or the like), then update [_process_match_param](https://github.com/balloob/home-assistant/blob/dev/homeassistant/helpers/event.py#L198) to parse cron syntax (*/5).\r\n\r\nIf you'd like automations to be able to be scheduled based off year, month, and day as well, then add that to automation's time platform.\r\n\r\nAs far as scheduling events for a specific time rather than checking with every time change, the next fire time could already be calculated and this could already be done. I don't think anyone has really justified a need for this however, as the current check is so fast and HASS already uses very low resources.\r\n\r\nI'm not sure that adding a dependency makes things easier to maintain. I'm also not convinced this really does much towards readability as everything is very clear already.\r\n\r\nWe could, with our current approach, provide a timestamp for the next time things will fire, but, again, I'm not sure a valid need for this has been presented.\r\n\r\nRight now, in my opinion, I think this adds more complexity than functionality."}}