{"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-pending","name":"comparison-pending","color":"fef2c0"},{"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/pending-signatures","name":"pending-signatures","color":"eb6420"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/reconstruction-pending","name":"reconstruction-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/simulation-pending","name":"simulation-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-pending","name":"tests-pending","color":"fbca04"}],"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":77,"closed_issues":803,"state":"open","created_at":"2015-06-23T07:27:04Z","updated_at":"2015-10-02T11:36:08Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2015-10-02T11:27:05Z","updated_at":"2015-10-02T11:36:12Z","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) by taking InOut iteration out of its merger. 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 will be addressed separately.\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/144995610","html_url":"https://github.com/cms-sw/cmssw/pull/11621#issuecomment-144995610","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/11621","id":144995610,"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":"2015-10-02T11:36:11Z","updated_at":"2015-10-02T11:36:11Z","body":"A new Pull Request was created by @makortel (Matti Kortelainen) for CMSSW_7_6_X.\n\nFixes for phase1 workflow\n\nIt involves the following packages:\n\nRecoTracker/FinalTrackSelectors\nRecoTracker/IterativeTracking\nSLHCUpgradeSimulations/Configuration\n\n\n@cmsbuild, @cvuosalo, @civanch, @mdhildreth, @slava77 can you please review it and eventually sign? Thanks.\n@ghellwig, @GiacomoSguazzoni, @rovere, @VinInn, @Martin-Grunewald, @mschrode, @istaslis, @gpetruc, @cerati, @dgulhan this is something you requested to watch as well.\nYou can sign-off by replying to this message having '+1' in the first line of your reply.\nYou can reject by replying  to this message having '-1' in the first line of your reply.\nIf you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.\n@degano you are the release manager for this.\nYou can merge this pull request by typing 'merge' in the first line of your comment.\n"}}