{"action":"created","issue":{"url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1615","labels_url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1615/labels{/name}","comments_url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1615/comments","events_url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1615/events","html_url":"https://github.com/TauLabs/TauLabs/pull/1615","id":76786496,"number":1615,"title":"OneShot: explicitly set the Sync mode and Resolution for Timers","user":{"login":"peabody124","id":618346,"avatar_url":"https://avatars.githubusercontent.com/u/618346?v=3","gravatar_id":"","url":"https://api.github.com/users/peabody124","html_url":"https://github.com/peabody124","followers_url":"https://api.github.com/users/peabody124/followers","following_url":"https://api.github.com/users/peabody124/following{/other_user}","gists_url":"https://api.github.com/users/peabody124/gists{/gist_id}","starred_url":"https://api.github.com/users/peabody124/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peabody124/subscriptions","organizations_url":"https://api.github.com/users/peabody124/orgs","repos_url":"https://api.github.com/users/peabody124/repos","events_url":"https://api.github.com/users/peabody124/events{/privacy}","received_events_url":"https://api.github.com/users/peabody124/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/TauLabs/TauLabs/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/TauLabs/TauLabs/labels/firmware","name":"firmware","color":"02e10c"},{"url":"https://api.github.com/repos/TauLabs/TauLabs/labels/gcs","name":"gcs","color":"d7e102"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2015-05-15T16:53:32Z","updated_at":"2015-05-17T17:45:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/TauLabs/TauLabs/pulls/1615","html_url":"https://github.com/TauLabs/TauLabs/pull/1615","diff_url":"https://github.com/TauLabs/TauLabs/pull/1615.diff","patch_url":"https://github.com/TauLabs/TauLabs/pull/1615.patch"},"body":"The previous code had a few issues. We could not independently set the resolution\r\nand the synchronization (to sensors) properties of channels. In addition, the settings\r\nwere per channel instead of per timer as the hardware specifies. This makes it consistent\r\nwith the update frequency properties of timers.\r\n\r\nSupport for OneShot configuration is also added to the setup wizard."},"comment":{"url":"https://api.github.com/repos/TauLabs/TauLabs/issues/comments/102826700","html_url":"https://github.com/TauLabs/TauLabs/pull/1615#issuecomment-102826700","issue_url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1615","id":102826700,"user":{"login":"peabody124","id":618346,"avatar_url":"https://avatars.githubusercontent.com/u/618346?v=3","gravatar_id":"","url":"https://api.github.com/users/peabody124","html_url":"https://github.com/peabody124","followers_url":"https://api.github.com/users/peabody124/followers","following_url":"https://api.github.com/users/peabody124/following{/other_user}","gists_url":"https://api.github.com/users/peabody124/gists{/gist_id}","starred_url":"https://api.github.com/users/peabody124/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peabody124/subscriptions","organizations_url":"https://api.github.com/users/peabody124/orgs","repos_url":"https://api.github.com/users/peabody124/repos","events_url":"https://api.github.com/users/peabody124/events{/privacy}","received_events_url":"https://api.github.com/users/peabody124/received_events","type":"User","site_admin":false},"created_at":"2015-05-17T17:45:24Z","updated_at":"2015-05-17T17:45:24Z","body":"@ernieift i like the idea of storing hte enum natively, but we need the actual divisor to use. That is a function of both the timer configuration _and_ the desired timer resolution. I don't see how to get it from just the enum value without adding another layer of logic into the set method. "}}