{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/11621","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/11621/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/11621/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/11621/events","html_url":"https://github.com/cms-sw/cmssw/pull/11621","id":109474703,"number":11621,"title":"Fixes for phase1 workflow","user":{"login":"makortel","id":33993,"avatar_url":"https://avatars.githubusercontent.com/u/33993?v=3","gravatar_id":"","url":"https://api.github.com/users/makortel","html_url":"https://github.com/makortel","followers_url":"https://api.github.com/users/makortel/followers","following_url":"https://api.github.com/users/makortel/following{/other_user}","gists_url":"https://api.github.com/users/makortel/gists{/gist_id}","starred_url":"https://api.github.com/users/makortel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/makortel/subscriptions","organizations_url":"https://api.github.com/users/makortel/orgs","repos_url":"https://api.github.com/users/makortel/repos","events_url":"https://api.github.com/users/makortel/events{/privacy}","received_events_url":"https://api.github.com/users/makortel/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-available","name":"comparison-available","color":"009800"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/fully-signed","name":"fully-signed","color":"009800"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/reconstruction-approved","name":"reconstruction-approved","color":"009800"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/simulation-approved","name":"simulation-approved","color":"009800"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-approved","name":"tests-approved","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/55","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_7_6_X","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/55/labels","id":1176788,"number":55,"title":"Next CMSSW_7_6_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":76,"closed_issues":833,"state":"open","created_at":"2015-06-23T07:27:04Z","updated_at":"2015-10-07T05:46:15Z","due_on":null,"closed_at":null},"comments":15,"created_at":"2015-10-02T11:27:05Z","updated_at":"2015-10-07T05:47:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/11621","html_url":"https://github.com/cms-sw/cmssw/pull/11621","diff_url":"https://github.com/cms-sw/cmssw/pull/11621.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/11621.patch"},"body":"This PR fixes issues in the phase1 workflow `runTheMatrix.py --what upgrade -l 10000`. The changes are mainly to recover from #10373, and to remove the remaining CASTOR modules in reconstruction sequence (I guessed from the existing removals that this is the intention for now).\r\n\r\nI also intentionally \"broke\" the \"displaced muon reconstruction\" (#7531) for Phase1 by taking InOut iteration out of its merger (technically it still should work but the results bad). It should be enabled again after the Phase1 tracking sequences are migrated to the new classifiers+mergers of #10373 (but I want to see the 76X-Phase1 tracking validated against 62X_SLHC Phase1 tracking before that).\r\n\r\nTogether with #11609 (GT update) this PR gets the reco step of upgrade wf 10000.0 to technically work on a DIGI step2.root produced with the earlier default GT (75X_upgrade2017_design_v1), but when using the same GT consistently, the reco step still fails. This failure is addressed separately in #11624.\r\n\r\nTested in 760pre6, no changes expected in Run1/2 workflows.\r\n\r\n@rovere @VinInn @boudoul @venturia"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/146082617","html_url":"https://github.com/cms-sw/cmssw/pull/11621#issuecomment-146082617","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/11621","id":146082617,"user":{"login":"davidlange6","id":5042883,"avatar_url":"https://avatars.githubusercontent.com/u/5042883?v=3","gravatar_id":"","url":"https://api.github.com/users/davidlange6","html_url":"https://github.com/davidlange6","followers_url":"https://api.github.com/users/davidlange6/followers","following_url":"https://api.github.com/users/davidlange6/following{/other_user}","gists_url":"https://api.github.com/users/davidlange6/gists{/gist_id}","starred_url":"https://api.github.com/users/davidlange6/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davidlange6/subscriptions","organizations_url":"https://api.github.com/users/davidlange6/orgs","repos_url":"https://api.github.com/users/davidlange6/repos","events_url":"https://api.github.com/users/davidlange6/events{/privacy}","received_events_url":"https://api.github.com/users/davidlange6/received_events","type":"User","site_admin":false},"created_at":"2015-10-07T05:47:50Z","updated_at":"2015-10-07T05:47:50Z","body":"+1\r\n"}}