{"action":"created","issue":{"url":"https://api.github.com/repos/GRIFFINCollaboration/GRSISort/issues/787","repository_url":"https://api.github.com/repos/GRIFFINCollaboration/GRSISort","labels_url":"https://api.github.com/repos/GRIFFINCollaboration/GRSISort/issues/787/labels{/name}","comments_url":"https://api.github.com/repos/GRIFFINCollaboration/GRSISort/issues/787/comments","events_url":"https://api.github.com/repos/GRIFFINCollaboration/GRSISort/issues/787/events","html_url":"https://github.com/GRIFFINCollaboration/GRSISort/issues/787","id":182089386,"number":787,"title":"Low gain vs high gain","user":{"login":"r3dunlop","id":5428160,"avatar_url":"https://avatars.githubusercontent.com/u/5428160?v=3","gravatar_id":"","url":"https://api.github.com/users/r3dunlop","html_url":"https://github.com/r3dunlop","followers_url":"https://api.github.com/users/r3dunlop/followers","following_url":"https://api.github.com/users/r3dunlop/following{/other_user}","gists_url":"https://api.github.com/users/r3dunlop/gists{/gist_id}","starred_url":"https://api.github.com/users/r3dunlop/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/r3dunlop/subscriptions","organizations_url":"https://api.github.com/users/r3dunlop/orgs","repos_url":"https://api.github.com/users/r3dunlop/repos","events_url":"https://api.github.com/users/r3dunlop/events{/privacy}","received_events_url":"https://api.github.com/users/r3dunlop/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/GRIFFINCollaboration/GRSISort/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/GRIFFINCollaboration/GRSISort/labels/Griffin","name":"Griffin","color":"eb6420"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-10-10T18:59:48Z","updated_at":"2016-10-10T20:54:26Z","closed_at":null,"body":"Right now the analysis tree building is setup to ignore the low gain events coming from the DAQ. For the current experiment this is not an issue. However, we should start thinking about how we want to construct our events when we have both useful high and low gain events since we do not want to correlate the same hit with itself. We are thinking we want to put these two output sensors into two different GRIFFIN hit vectors. We can then add a `GetLowGainHit()` function to return the low gain hits and a `GetLowGainMultiplicity()` etc..."},"comment":{"url":"https://api.github.com/repos/GRIFFINCollaboration/GRSISort/issues/comments/252743420","html_url":"https://github.com/GRIFFINCollaboration/GRSISort/issues/787#issuecomment-252743420","issue_url":"https://api.github.com/repos/GRIFFINCollaboration/GRSISort/issues/787","id":252743420,"user":{"login":"MikeBowry","id":14794662,"avatar_url":"https://avatars.githubusercontent.com/u/14794662?v=3","gravatar_id":"","url":"https://api.github.com/users/MikeBowry","html_url":"https://github.com/MikeBowry","followers_url":"https://api.github.com/users/MikeBowry/followers","following_url":"https://api.github.com/users/MikeBowry/following{/other_user}","gists_url":"https://api.github.com/users/MikeBowry/gists{/gist_id}","starred_url":"https://api.github.com/users/MikeBowry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MikeBowry/subscriptions","organizations_url":"https://api.github.com/users/MikeBowry/orgs","repos_url":"https://api.github.com/users/MikeBowry/repos","events_url":"https://api.github.com/users/MikeBowry/events{/privacy}","received_events_url":"https://api.github.com/users/MikeBowry/received_events","type":"User","site_admin":false},"created_at":"2016-10-10T20:54:26Z","updated_at":"2016-10-10T20:54:26Z","body":"This is a good point - splitting low gain and high gain hits into two different vectors sounds like a reasonable solution. How are these hits differentiated - from the channel mnemonic? So right now if I sort and analysis tree, `GetMultiplicity()` only returns one set of hits? Does the same apply to other functions such as `GetArrayNumber()`?\r\nLots of questions, sorry."}}
{"actor":{"display_login":"MikeBowry"}}