{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/13945","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/13945/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/13945/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/13945/events","html_url":"https://github.com/cms-sw/cmssw/pull/13945","id":146205863,"number":13945,"title":"era-based run1 tracking setup and T0 Data processing update","user":{"login":"slava77","id":4676718,"avatar_url":"https://avatars.githubusercontent.com/u/4676718?v=3","gravatar_id":"","url":"https://api.github.com/users/slava77","html_url":"https://github.com/slava77","followers_url":"https://api.github.com/users/slava77/followers","following_url":"https://api.github.com/users/slava77/following{/other_user}","gists_url":"https://api.github.com/users/slava77/gists{/gist_id}","starred_url":"https://api.github.com/users/slava77/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/slava77/subscriptions","organizations_url":"https://api.github.com/users/slava77/orgs","repos_url":"https://api.github.com/users/slava77/repos","events_url":"https://api.github.com/users/slava77/events{/privacy}","received_events_url":"https://api.github.com/users/slava77/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/alca-pending","name":"alca-pending","color":"fbca04"},{"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/dqm-approved","name":"dqm-approved","color":"009800"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/operations-pending","name":"operations-pending","color":"fbca04"},{"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/tests-approved","name":"tests-approved","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/57","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_8_0_X","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/57/labels","id":1362157,"number":57,"title":"Next CMSSW_8_0_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":58,"closed_issues":936,"state":"open","created_at":"2015-10-18T08:35:44Z","updated_at":"2016-04-08T11:24:37Z","due_on":null,"closed_at":null},"comments":33,"created_at":"2016-04-06T06:53:49Z","updated_at":"2016-04-08T18:43:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/13945","html_url":"https://github.com/cms-sw/cmssw/pull/13945","diff_url":"https://github.com/cms-sw/cmssw/pull/13945.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/13945.patch"},"body":"- Run2_2016_trackingLowPU  is added by @makortel to replace functionality earlier available from RecoTracker/Configuration/customiseForRunI.customiseForRunI\r\n\r\nDataProcessing setup has a matching ppEra_Run2_2016_trackingLowPU scenario\r\n\r\nThis PR also includes an attempt to revive the setup with old customization function: it is available with pplowpuEra_Run2_2016 data processing scenario. However, the customization is out of date and while the code configures, it doesn't run.\r\nWith the customization, detachedTripletStepChi2Est  is  a Chi2MeasurementEstimatorESProducer (compared to Chi2ChargeMeasurementEstimatorESProducer with the era). The PSet for the former requires no clusterChargeCut is present and PSet validation fails.\r\n\r\n@makortel @VinInn @rovere  in case there is some need to support this customization function, it needs an update.\r\nA better choice is to live with the era-based setup.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/207551822","html_url":"https://github.com/cms-sw/cmssw/pull/13945#issuecomment-207551822","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/13945","id":207551822,"user":{"login":"slava77","id":4676718,"avatar_url":"https://avatars.githubusercontent.com/u/4676718?v=3","gravatar_id":"","url":"https://api.github.com/users/slava77","html_url":"https://github.com/slava77","followers_url":"https://api.github.com/users/slava77/followers","following_url":"https://api.github.com/users/slava77/following{/other_user}","gists_url":"https://api.github.com/users/slava77/gists{/gist_id}","starred_url":"https://api.github.com/users/slava77/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/slava77/subscriptions","organizations_url":"https://api.github.com/users/slava77/orgs","repos_url":"https://api.github.com/users/slava77/repos","events_url":"https://api.github.com/users/slava77/events{/privacy}","received_events_url":"https://api.github.com/users/slava77/received_events","type":"User","site_admin":false},"created_at":"2016-04-08T18:43:04Z","updated_at":"2016-04-08T18:43:04Z","body":"+1\r\n\r\nfor https://github.com/cms-sw/cmssw/pull/13945\t0004500\r\n- jenkins tests pass and comparisons with baseline show no differences as expected for existing workflows\r\n- tested in CMSSW_8_0_3_patch1\r\n    - see notes above for the effect on complete detector configuration\r\n- T0 should use ppEra_Run2_2016_trackingLowPU for this purpose (early stable collision period with 0T and no pixel tracker)\r\n- specific behavior is tested using MinimumBias 245200 file (22C6063B-C7FF-E411-8793-02163E01425D.root ) 2000 events, a run with B=0T and no pixel: the behavior is as expected\r\n![all_ppera_run2_2016_trackinglowpuvsppera_run2_2016_minimumbias245200c_globalmuontracks_pt](https://cloud.githubusercontent.com/assets/4676718/14393834/ffb09804-fd96-11e5-984f-678b778e8c7b.png)\r\n![all_ppera_run2_2016_trackinglowpuvsppera_run2_2016_minimumbias245200c_log10recogsftracks_electrongsftracks__reco_obj_pt](https://cloud.githubusercontent.com/assets/4676718/14393841/07cb3404-fd97-11e5-9b0d-12b2aecf893a.png)\r\n\r\n![all_ppera_run2_2016_trackinglowpuvsppera_run2_2016_minimumbias245200c_log10recotracks_generaltracks__reco_obj_pt](https://cloud.githubusercontent.com/assets/4676718/14393858/2319de7c-fd97-11e5-8707-d2ee852429c7.png)\r\n\r\n- short tests were made with DataProcessing scripts as well\r\n```\r\npython $CMSSW_BASE/src/Configuration/DataProcessing/test/RunExpressProcessing.py --scenario ppEra_Run2_2016_trackingLowPU --fevt --dqmio --global-tag 80X_dataRun2_Express_v4 --lfn=file:22C6063B-C7FF-E411-8793-02163E01425D.root  --alcareco SiStripPCLHistos+SiStripCalZeroBias+SiStripCalMinBias+TkAlMinBias+DtCalib+PromptCalibProd+Hotline+PromptCalibProdSiStrip+PromptCalibProdSiStripGains+LumiPixelsMinBias \r\n```\r\nand\r\n```\r\npython $CMSSW_BASE/src/Configuration/DataProcessing/test/RunPromptReco.py --scenario ppEra_Run2_2016_trackingLowPU --reco --aod --miniaod --dqmio --global-tag 80X_dataRun2_Prompt_v4 --lfn=file:22C6063B-C7FF-E411-8793-02163E01425D.root  --alcareco SiStripCalZeroBias+SiStripCalMinBias+TkAlMinBias  --dqmSeq @common --PhysicsSkim=LogError+LogErrorMonitor\r\n```"}}