{"action":"created","issue":{"url":"https://api.github.com/repos/sinara-hw/Booster/issues/73","repository_url":"https://api.github.com/repos/sinara-hw/Booster","labels_url":"https://api.github.com/repos/sinara-hw/Booster/issues/73/labels{/name}","comments_url":"https://api.github.com/repos/sinara-hw/Booster/issues/73/comments","events_url":"https://api.github.com/repos/sinara-hw/Booster/issues/73/events","html_url":"https://github.com/sinara-hw/Booster/issues/73","id":356799254,"node_id":"MDU6SXNzdWUzNTY3OTkyNTQ=","number":73,"title":"output power hardware interlock","user":{"login":"hartytp","id":21218399,"node_id":"MDQ6VXNlcjIxMjE4Mzk5","avatar_url":"https://avatars0.githubusercontent.com/u/21218399?v=4","gravatar_id":"","url":"https://api.github.com/users/hartytp","html_url":"https://github.com/hartytp","followers_url":"https://api.github.com/users/hartytp/followers","following_url":"https://api.github.com/users/hartytp/following{/other_user}","gists_url":"https://api.github.com/users/hartytp/gists{/gist_id}","starred_url":"https://api.github.com/users/hartytp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hartytp/subscriptions","organizations_url":"https://api.github.com/users/hartytp/orgs","repos_url":"https://api.github.com/users/hartytp/repos","events_url":"https://api.github.com/users/hartytp/events{/privacy}","received_events_url":"https://api.github.com/users/hartytp/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":32,"created_at":"2018-09-04T13:01:42Z","updated_at":"2018-09-07T18:11:32Z","closed_at":null,"author_association":"COLLABORATOR","body":"IMHO the main function of the hardware interlock is to protect the load, not the amplifier. So, if it has a fixed value then it's not much use.\r\n\r\nIf the comparator isn't as accurate as the voltage output after calibration then we have two options for the next hardware revision: (a) scrap the hardware interlock; (b) add an external comparator\r\n\r\nI'd prefer to go for (b) as I think that having a fast hardware interlock is very valuable to protect sensitive loads.\r\n\r\nIn the mean time, I'd be curious to test out how accurate the hardware interlock is after calibration using the current hardware revision to see if this is really a problem!"},"comment":{"url":"https://api.github.com/repos/sinara-hw/Booster/issues/comments/419522226","html_url":"https://github.com/sinara-hw/Booster/issues/73#issuecomment-419522226","issue_url":"https://api.github.com/repos/sinara-hw/Booster/issues/73","id":419522226,"node_id":"MDEyOklzc3VlQ29tbWVudDQxOTUyMjIyNg==","user":{"login":"wizath","id":11598020,"node_id":"MDQ6VXNlcjExNTk4MDIw","avatar_url":"https://avatars2.githubusercontent.com/u/11598020?v=4","gravatar_id":"","url":"https://api.github.com/users/wizath","html_url":"https://github.com/wizath","followers_url":"https://api.github.com/users/wizath/followers","following_url":"https://api.github.com/users/wizath/following{/other_user}","gists_url":"https://api.github.com/users/wizath/gists{/gist_id}","starred_url":"https://api.github.com/users/wizath/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wizath/subscriptions","organizations_url":"https://api.github.com/users/wizath/orgs","repos_url":"https://api.github.com/users/wizath/repos","events_url":"https://api.github.com/users/wizath/events{/privacy}","received_events_url":"https://api.github.com/users/wizath/received_events","type":"User","site_admin":false},"created_at":"2018-09-07T18:11:32Z","updated_at":"2018-09-07T18:11:32Z","author_association":"COLLABORATOR","body":"First: 4095 is way too much for output interlock. I'm starting calibration from about 1500's and decrease value till interlock is tripped. That DAC is 12 bit only so 65535 should get 0V, so interlock will trip instantly.\r\n\r\nIf you type stop that will stop interlock task (leds updating - need to remove that). You have to type start each time. Will fix that in next release"}}
{"actor":{"display_login":"wizath"}}