{"action":"created","issue":{"url":"https://api.github.com/repos/jiafulow/L1TriggerSep2016/issues/17","repository_url":"https://api.github.com/repos/jiafulow/L1TriggerSep2016","labels_url":"https://api.github.com/repos/jiafulow/L1TriggerSep2016/issues/17/labels{/name}","comments_url":"https://api.github.com/repos/jiafulow/L1TriggerSep2016/issues/17/comments","events_url":"https://api.github.com/repos/jiafulow/L1TriggerSep2016/issues/17/events","html_url":"https://github.com/jiafulow/L1TriggerSep2016/pull/17","id":211821697,"number":17,"title":"Fix data-emulator agreement","user":{"login":"jiafulow","id":5023845,"avatar_url":"https://avatars2.githubusercontent.com/u/5023845?v=3","gravatar_id":"","url":"https://api.github.com/users/jiafulow","html_url":"https://github.com/jiafulow","followers_url":"https://api.github.com/users/jiafulow/followers","following_url":"https://api.github.com/users/jiafulow/following{/other_user}","gists_url":"https://api.github.com/users/jiafulow/gists{/gist_id}","starred_url":"https://api.github.com/users/jiafulow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jiafulow/subscriptions","organizations_url":"https://api.github.com/users/jiafulow/orgs","repos_url":"https://api.github.com/users/jiafulow/repos","events_url":"https://api.github.com/users/jiafulow/events{/privacy}","received_events_url":"https://api.github.com/users/jiafulow/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-03-03T22:40:54Z","updated_at":"2017-03-14T21:13:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jiafulow/L1TriggerSep2016/pulls/17","html_url":"https://github.com/jiafulow/L1TriggerSep2016/pull/17","diff_url":"https://github.com/jiafulow/L1TriggerSep2016/pull/17.diff","patch_url":"https://github.com/jiafulow/L1TriggerSep2016/pull/17.patch"},"body":"This PR contains the necessary fixes to bring the data-emulator agreement to >99.5%. It includes:\r\n- Implementation of FW sorting for CSC hits\r\n- Implementation of ME1/1 theta duplication in FW\r\n- Minor fixes like ME1/1 theta initialization bug, precise theta assignment, zone numbering, etc\r\n- Additional options like 'BugSameSectorPt0'\r\n\r\nThis PR should be merged after PR #16.\r\n\r\nThis PR should close issue #12."},"comment":{"url":"https://api.github.com/repos/jiafulow/L1TriggerSep2016/issues/comments/286562667","html_url":"https://github.com/jiafulow/L1TriggerSep2016/pull/17#issuecomment-286562667","issue_url":"https://api.github.com/repos/jiafulow/L1TriggerSep2016/issues/17","id":286562667,"user":{"login":"jiafulow","id":5023845,"avatar_url":"https://avatars2.githubusercontent.com/u/5023845?v=3","gravatar_id":"","url":"https://api.github.com/users/jiafulow","html_url":"https://github.com/jiafulow","followers_url":"https://api.github.com/users/jiafulow/followers","following_url":"https://api.github.com/users/jiafulow/following{/other_user}","gists_url":"https://api.github.com/users/jiafulow/gists{/gist_id}","starred_url":"https://api.github.com/users/jiafulow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jiafulow/subscriptions","organizations_url":"https://api.github.com/users/jiafulow/orgs","repos_url":"https://api.github.com/users/jiafulow/repos","events_url":"https://api.github.com/users/jiafulow/events{/privacy}","received_events_url":"https://api.github.com/users/jiafulow/received_events","type":"User","site_admin":false},"created_at":"2017-03-14T21:13:55Z","updated_at":"2017-03-14T21:13:55Z","body":"Using Run 281707 (2016H), 20,314 events in the ExpressPhysics primary dataset, the level of disagreement is 8/20,314 = 0.04%. All of them have BX mismatched and appear to have identical LCTs received in consecutive BX's.\r\n\r\nRunning over more events in the same dataset, the level of disagreement is 598/859,744 = 0.07%. BX mismatch accounts for 0.04%. The rest is due to mismatches in number of tracks, track mode, and pT address. These events have not been diagnosed.\r\n"}}
{"actor":{"display_login":"jiafulow"}}