{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/24294","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/24294/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/24294/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/24294/events","html_url":"https://github.com/cms-sw/cmssw/pull/24294","id":350815646,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA4NTY3NDkx","number":24294,"title":"Changed CSC binary examiner mask to enable ALCT CRC checks - CMSSW_10_2_X","user":{"login":"barvic","id":5807200,"node_id":"MDQ6VXNlcjU4MDcyMDA=","avatar_url":"https://avatars1.githubusercontent.com/u/5807200?v=4","gravatar_id":"","url":"https://api.github.com/users/barvic","html_url":"https://github.com/barvic","followers_url":"https://api.github.com/users/barvic/followers","following_url":"https://api.github.com/users/barvic/following{/other_user}","gists_url":"https://api.github.com/users/barvic/gists{/gist_id}","starred_url":"https://api.github.com/users/barvic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/barvic/subscriptions","organizations_url":"https://api.github.com/users/barvic/orgs","repos_url":"https://api.github.com/users/barvic/repos","events_url":"https://api.github.com/users/barvic/events{/privacy}","received_events_url":"https://api.github.com/users/barvic/received_events","type":"User","site_admin":false},"labels":[{"id":425669377,"node_id":"MDU6TGFiZWw0MjU2NjkzNzc=","url":"https://api.github.com/repos/cms-sw/cmssw/labels/backport","name":"backport","color":"0000ff","default":false},{"id":88653795,"node_id":"MDU6TGFiZWw4ODY1Mzc5NQ==","url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fbca04","default":false},{"id":56953091,"node_id":"MDU6TGFiZWw1Njk1MzA5MQ==","url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04","default":false},{"id":53167770,"node_id":"MDU6TGFiZWw1MzE2Nzc3MA==","url":"https://api.github.com/repos/cms-sw/cmssw/labels/pending-signatures","name":"pending-signatures","color":"ff8000","default":false},{"id":52820010,"node_id":"MDU6TGFiZWw1MjgyMDAxMA==","url":"https://api.github.com/repos/cms-sw/cmssw/labels/reconstruction-pending","name":"reconstruction-pending","color":"fbca04","default":false},{"id":170521617,"node_id":"MDU6TGFiZWwxNzA1MjE2MTc=","url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-started","name":"tests-started","color":"ff8000","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/76","html_url":"https://github.com/cms-sw/cmssw/milestone/76","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/76/labels","id":3227631,"node_id":"MDk6TWlsZXN0b25lMzIyNzYzMQ==","number":76,"title":"CMSSW_10_2_X","description":null,"creator":{"login":"smuzaffar","id":4115138,"node_id":"MDQ6VXNlcjQxMTUxMzg=","avatar_url":"https://avatars3.githubusercontent.com/u/4115138?v=4","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":27,"closed_issues":767,"state":"open","created_at":"2018-03-29T07:57:10Z","updated_at":"2018-08-16T12:42:35Z","due_on":null,"closed_at":null},"comments":4,"created_at":"2018-08-15T13:48:12Z","updated_at":"2018-08-16T15:36:17Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/24294","html_url":"https://github.com/cms-sw/cmssw/pull/24294","diff_url":"https://github.com/cms-sw/cmssw/pull/24294.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/24294.patch"},"body":"EventFilter/CSCRawToDigi - Changed CSC binary examiner mask to enable ALCT CRC checks. \r\nIt should prevent passing of corrupted (bad ALCT) CSC event data to main CSC data unpacking code. \r\nRelated to request from HLT group, which was investigating random HLT crashes in run 320917"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/413588633","html_url":"https://github.com/cms-sw/cmssw/pull/24294#issuecomment-413588633","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/24294","id":413588633,"node_id":"MDEyOklzc3VlQ29tbWVudDQxMzU4ODYzMw==","user":{"login":"perrotta","id":4069749,"node_id":"MDQ6VXNlcjQwNjk3NDk=","avatar_url":"https://avatars1.githubusercontent.com/u/4069749?v=4","gravatar_id":"","url":"https://api.github.com/users/perrotta","html_url":"https://github.com/perrotta","followers_url":"https://api.github.com/users/perrotta/followers","following_url":"https://api.github.com/users/perrotta/following{/other_user}","gists_url":"https://api.github.com/users/perrotta/gists{/gist_id}","starred_url":"https://api.github.com/users/perrotta/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/perrotta/subscriptions","organizations_url":"https://api.github.com/users/perrotta/orgs","repos_url":"https://api.github.com/users/perrotta/repos","events_url":"https://api.github.com/users/perrotta/events{/privacy}","received_events_url":"https://api.github.com/users/perrotta/received_events","type":"User","site_admin":false},"created_at":"2018-08-16T15:36:17Z","updated_at":"2018-08-16T15:36:17Z","author_association":"CONTRIBUTOR","body":"Thank you @barvic for the explanations you provided in https://github.com/cms-sw/cmssw/pull/24295#issuecomment-413550040 .\r\n\r\nAbout your last point [1], could you please elaborate here a bit more? If we merge this backport, we should understand whether any difference with the previous recorded data is expected, or if it is really only on the handling of corrupted data. Do you have any slide or presentation that explains what we should expect in detail?\r\n\r\n[1]\r\n\"I don't expect effects on normal non-corrupted data. Only expected effect is additional unpacking rejection of corrupted data for this particular rare type of detected errors, which should be handled this way for the unpacking stability purposes initially\". "}}
{"actor":{"display_login":"perrotta"}}