21.9 - Fixing cuts to be compatible with ITk Inclined Duals in 20.20.10.7
While checking the loss in track efficiency, I have compared the log files obtained in 20.20.10.7 for the Inclined Duals reconstruction and I have found a mistake in the configured reconstruction cuts in 21.9. My bad, I guess I have copied the wrong configuration and I didn't figured that out so far. You can check the diff of this MR.
I have run the test comparing 20.20.10.7 against 21.9 with the correct cuts and the situation improves: https://test-dcube.web.cern.ch/test-dcube/physval_checkCuts.root.dcube.xml.php
I guess we should as well update the vertexing cuts which are not important for single muon jobs. Below you can find the cut configuration as obtained in 21.9 vs 20.20.10.7:
In 21.9:
****** Inner Detector Primary Vertex Reconstruction Cuts ****************************
*
* mode is : SLHC
* cut level for primary vertexing is : 3
*
* Track Cut Level : TightPrimary
*
* min pT : 500.0 MeV
* Max IP d0 : 1.0
* Max IP Z0 : 1000.0
* Max z0 : 1000.0
* Max Sig IP d0 : 0.35
* Max Sig IP z0 : 2.5
* Max Sig d0 : 4.0
* Max Sig z0 : -1.0
* Max eta : 9999.0
* Use TrackSummary : True
* Min N Innermost Layer Hits : 0
* Min N Pix Hit : 1
* Max N Pix Holes : 1
* Min N SCT Hits : 5
* Min N Si Hits : 7
* Min N TRT Hits : 0
* Min N high threshold hits TRT : 1
* Min N high threshold hits TRT (incl. outliers) : 1
* Use shared hit info : False
* Use Track Quality info : True
* Chi2/ndf of the fit : 3.5
* TRT Max eta : 1.9
*
* Chi2 cut method : 2
* Enable Multiple Vertices : True
*
************************************************************************************
and in 20.20.10.7:
****** Inner Detector Primary Vertex Reconstruction Cuts ****************************
*
* mode is : SLHC
* cut level for primary vertexing is : 3
*
* Track Cut Level : TightPrimary
*
* min pT : 900.0 MeV
* Max IP d0 : 1.0
* Max IP Z0 : 1000.0
* Max z0 : 1000.0
* Max Sig IP d0 : 0.35
* Max Sig IP z0 : 2.5
* Max Sig d0 : 4.0
* Max Sig z0 : -1.0
* Max eta : 9999.0
* Use TrackSummary : True
* Min N Innermost Layer Hits : 0
* Min N Pix Hit : 3
* Max N Pix Holes : 1
* Min N SCT Hits : 0
* Min N Si Hits : 7
* Min N TRT Hits : 0
* Min N high threshold hits TRT : 1
* Min N high threshold hits TRT (incl. outliers) : 1
* Use shared hit info : False
* Use Track Quality info : True
* Chi2/ndf of the fit : 3.5
* TRT Max eta : 1.9
*
* Chi2 cut method : 2
* Enable Multiple Vertices : True
*
************************************************************************************
Can others please comment before removing the WIP prefix? @nstyles, @asalzbur, @xai, @npetters, @lmijovic, @goblirsc, @swaban
Merge request reports
Activity
Hi @ncalace,
Great that you found this! The efficiency indeed looks much better, so no objections from my side to moving this out of WIP.
Regarding the vertexing cuts, I guess we can just take the 20.20 cuts directly for the moment. These are 'flat' cuts with no eta track eta dependence?
added InnerDetector review-pending-level-1 labels
mentioned in merge request !20726 (merged)
CI Result SUCCESSAthena externals cmake make required tests optional tests Full details available at NICOS MR-20711-2019-01-29-13-56
Athena: number of compilation errors 0, warnings 166
CI Jenkins server is switched to https://atlas-sit-ci.cern.ch. It is accessible world-wide (behind CERN SSO). In old links to Jenkins server aibuild080.cern.ch:8080 should be replaced with atlas-sit-ci.cern.ch For experts only: Jenkins output [CI-MERGE-REQUEST 32942]added review-approved label and removed review-pending-level-1 label
mentioned in commit 8bfdb041