{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/16302","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16302/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16302/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16302/events","html_url":"https://github.com/cms-sw/cmssw/pull/16302","id":184487654,"number":16302,"title":"Fix crosstalk","user":{"login":"kpedro88","id":4672808,"avatar_url":"https://avatars.githubusercontent.com/u/4672808?v=3","gravatar_id":"","url":"https://api.github.com/users/kpedro88","html_url":"https://github.com/kpedro88","followers_url":"https://api.github.com/users/kpedro88/followers","following_url":"https://api.github.com/users/kpedro88/following{/other_user}","gists_url":"https://api.github.com/users/kpedro88/gists{/gist_id}","starred_url":"https://api.github.com/users/kpedro88/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kpedro88/subscriptions","organizations_url":"https://api.github.com/users/kpedro88/orgs","repos_url":"https://api.github.com/users/kpedro88/repos","events_url":"https://api.github.com/users/kpedro88/events{/privacy}","received_events_url":"https://api.github.com/users/kpedro88/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-10-21T13:41:11Z","updated_at":"2016-10-21T13:59:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/16302","html_url":"https://github.com/cms-sw/cmssw/pull/16302","diff_url":"https://github.com/cms-sw/cmssw/pull/16302.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/16302.patch"},"body":"It turns out that the measured gain for SiPMs in fC/GeV includes the crosstalk. Therefore, when this gain is divided out of the Geant4 energy in the simulation to calculate the number of photoelectrons, the crosstalk should also be divided out. We model the crosstalk with a Borel-Tanner distribution which has mean 1/(1-lambda).\r\n\r\n@mariadalfonso showed that this fixes the single pion response: https://dalfonso.web.cern.ch/dalfonso/HCAL/isoTrack/AbsoluteHCALResponse_afterfixes.pdf"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/255384485","html_url":"https://github.com/cms-sw/cmssw/pull/16302#issuecomment-255384485","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16302","id":255384485,"user":{"login":"kpedro88","id":4672808,"avatar_url":"https://avatars.githubusercontent.com/u/4672808?v=3","gravatar_id":"","url":"https://api.github.com/users/kpedro88","html_url":"https://github.com/kpedro88","followers_url":"https://api.github.com/users/kpedro88/followers","following_url":"https://api.github.com/users/kpedro88/following{/other_user}","gists_url":"https://api.github.com/users/kpedro88/gists{/gist_id}","starred_url":"https://api.github.com/users/kpedro88/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kpedro88/subscriptions","organizations_url":"https://api.github.com/users/kpedro88/orgs","repos_url":"https://api.github.com/users/kpedro88/repos","events_url":"https://api.github.com/users/kpedro88/events{/privacy}","received_events_url":"https://api.github.com/users/kpedro88/received_events","type":"User","site_admin":false},"created_at":"2016-10-21T13:59:30Z","updated_at":"2016-10-21T13:59:30Z","body":"@smuzaffar can you prod the bot?"}}
{"actor":{"display_login":"kpedro88"}}