{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/15959","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/15959/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/15959/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/15959/events","html_url":"https://github.com/cms-sw/cmssw/pull/15959","id":178649290,"number":15959,"title":"HBHE: siPM M2 reco fixes ","user":{"login":"mariadalfonso","id":5656714,"avatar_url":"https://avatars.githubusercontent.com/u/5656714?v=3","gravatar_id":"","url":"https://api.github.com/users/mariadalfonso","html_url":"https://github.com/mariadalfonso","followers_url":"https://api.github.com/users/mariadalfonso/followers","following_url":"https://api.github.com/users/mariadalfonso/following{/other_user}","gists_url":"https://api.github.com/users/mariadalfonso/gists{/gist_id}","starred_url":"https://api.github.com/users/mariadalfonso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mariadalfonso/subscriptions","organizations_url":"https://api.github.com/users/mariadalfonso/orgs","repos_url":"https://api.github.com/users/mariadalfonso/repos","events_url":"https://api.github.com/users/mariadalfonso/events{/privacy}","received_events_url":"https://api.github.com/users/mariadalfonso/received_events","type":"User","site_admin":false},"labels":[{"id":88653795,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fbca04","default":false},{"id":53149121,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/fully-signed","name":"fully-signed","color":"009800","default":false},{"id":56953091,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04","default":false},{"id":52882429,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/reconstruction-approved","name":"reconstruction-approved","color":"009800","default":false},{"id":170521617,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-started","name":"tests-started","color":"eb6420","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/59","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_8_1_X","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/59/labels","id":1590252,"number":59,"title":"Next CMSSW_8_1_X","description":"","creator":{"login":"smuzaffar","id":4115138,"avatar_url":"https://avatars.githubusercontent.com/u/4115138?v=3","gravatar_id":"","url":"https://api.github.com/users/smuzaffar","html_url":"https://github.com/smuzaffar","followers_url":"https://api.github.com/users/smuzaffar/followers","following_url":"https://api.github.com/users/smuzaffar/following{/other_user}","gists_url":"https://api.github.com/users/smuzaffar/gists{/gist_id}","starred_url":"https://api.github.com/users/smuzaffar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smuzaffar/subscriptions","organizations_url":"https://api.github.com/users/smuzaffar/orgs","repos_url":"https://api.github.com/users/smuzaffar/repos","events_url":"https://api.github.com/users/smuzaffar/events{/privacy}","received_events_url":"https://api.github.com/users/smuzaffar/received_events","type":"User","site_admin":false},"open_issues":66,"closed_issues":1782,"state":"open","created_at":"2016-02-18T09:12:12Z","updated_at":"2016-10-28T16:52:46Z","due_on":"2016-11-01T00:00:00Z","closed_at":null},"comments":176,"created_at":"2016-09-22T15:55:38Z","updated_at":"2016-10-28T18:56:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/15959","html_url":"https://github.com/cms-sw/cmssw/pull/15959","diff_url":"https://github.com/cms-sw/cmssw/pull/15959.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/15959.patch"},"body":"1) FIX the initialization of the siPM pulse (Run2-phase1). \r\nBefore the M2 siPM fits was done with the HPD pulse and parameters.\r\nThe  save 20% of CPU time on single particle no PU\r\n2) Add extra features: add the ADC granularity or siPM (phase1)\r\n3) fix the dark current term in the noise term of the M2-chi2\r\n(http://dalfonso.web.cern.ch/dalfonso/DarkCurrentOnSiPM_PR15959.pdf)\r\n\r\nThis PR discuss also a possible move the initial time of the M2 fit to 2ns (Run2-phase1 ), but this is removed from the list of commit.\r\nThe save an extra 20% of CPU time on single particle no PU\r\n\r\nSome reference slide\r\nhttps://indico.cern.ch/event/570231/contributions/2306435/attachments/1341696/2021200/PR15959.pdf\r\n"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/257000750","html_url":"https://github.com/cms-sw/cmssw/pull/15959#issuecomment-257000750","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/15959","id":257000750,"user":{"login":"cmsbuild","id":4170616,"avatar_url":"https://avatars.githubusercontent.com/u/4170616?v=3","gravatar_id":"","url":"https://api.github.com/users/cmsbuild","html_url":"https://github.com/cmsbuild","followers_url":"https://api.github.com/users/cmsbuild/followers","following_url":"https://api.github.com/users/cmsbuild/following{/other_user}","gists_url":"https://api.github.com/users/cmsbuild/gists{/gist_id}","starred_url":"https://api.github.com/users/cmsbuild/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmsbuild/subscriptions","organizations_url":"https://api.github.com/users/cmsbuild/orgs","repos_url":"https://api.github.com/users/cmsbuild/repos","events_url":"https://api.github.com/users/cmsbuild/events{/privacy}","received_events_url":"https://api.github.com/users/cmsbuild/received_events","type":"User","site_admin":false},"created_at":"2016-10-28T18:56:37Z","updated_at":"2016-10-28T18:56:37Z","body":"Comparison job queued."}}
{"actor":{"display_login":"cmsbuild"}}