{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/17751","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/17751/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/17751/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/17751/events","html_url":"https://github.com/cms-sw/cmssw/pull/17751","id":211423501,"number":17751,"title":"fix sequences for HE plan1","user":{"login":"kpedro88","id":4672808,"avatar_url":"https://avatars3.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":[{"id":88653795,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fbca04","default":false},{"id":52881747,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/operations-pending","name":"operations-pending","color":"fbca04","default":false},{"id":56953091,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04","default":false},{"id":53167770,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/pending-signatures","name":"pending-signatures","color":"eb6420","default":false},{"id":52820010,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/reconstruction-pending","name":"reconstruction-pending","color":"fbca04","default":false},{"id":56131152,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-pending","name":"tests-pending","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/65","html_url":"https://github.com/cms-sw/cmssw/milestone/65","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/65/labels","id":2358370,"number":65,"title":"Next CMSSW_9_1_X","description":"","creator":{"login":"smuzaffar","id":4115138,"avatar_url":"https://avatars0.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":60,"closed_issues":9,"state":"open","created_at":"2017-03-02T09:15:47Z","updated_at":"2017-03-02T15:26:23Z","due_on":null,"closed_at":null},"comments":2,"created_at":"2017-03-02T15:25:09Z","updated_at":"2017-03-02T15:26:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/17751","html_url":"https://github.com/cms-sw/cmssw/pull/17751","diff_url":"https://github.com/cms-sw/cmssw/pull/17751.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/17751.patch"},"body":"The \"final\" set of HCAL RecHits produced needs to be named `hbhereco` in order for downstream modules to pick up the right collection. This means that, in sequences where `HBHEIsolatedNoiseReflagger` is not run, the `hbheplan1` module needs to be renamed to `hbhereco`. Therefore, where the `HBHEPhase1Reconstructor` would normally be named `hbhereco` in these sequences, it needs to be `hbheprereco` instead.\r\n\r\nThis change is made for NZS and Cosmic sequences. It should hopefully fix the crash at the Tier0. This PR will be backported to 90X and 84X (once the fix is agreed to work).\r\n\r\n@bsunanda: at some later date we need to make AlCa sequences aware of Phase1 changes:\r\nhttps://github.com/cms-sw/cmssw/blob/master/Calibration/HcalAlCaRecoProducers/python/ALCARECOHcalCalMinBias_cff.py\r\nhttps://github.com/cms-sw/cmssw/blob/master/Calibration/HcalAlCaRecoProducers/python/ALCARECOHcalCalPedestal_cff.py\r\n(I just noticed today that these exist.)\r\n\r\nattn: @deguio, @hatakeyamak, @abdoulline\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/283684287","html_url":"https://github.com/cms-sw/cmssw/pull/17751#issuecomment-283684287","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/17751","id":283684287,"user":{"login":"cmsbuild","id":4170616,"avatar_url":"https://avatars2.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":"2017-03-02T15:26:37Z","updated_at":"2017-03-02T15:26:37Z","body":"A new Pull Request was created by  @kpedro88 (Kevin Pedro) for master.\n\nIt involves the following packages:\n\nConfiguration/StandardSequences\nRecoLocalCalo/Configuration\n\n\n@cmsbuild, @cvuosalo, @franzoni, @slava77, @davidlange6 can you please review it and eventually sign? Thanks.\n@ghellwig, @makortel, @felicepantaleo, @GiacomoSguazzoni, @rovere, @VinInn, @Martin-Grunewald, @dgulhan this is something you requested to watch as well.\n@muzaffar, @davidlange6, @smuzaffar you are the release manager for this.\n\ncms-bot commands are listed here cms-sw/cmssw#13028\n"}}
{"actor":{"display_login":"cmsbuild"}}