Skip to content

Fix dE/dx track and hit-based displaced vertex trigger: ATR-28087

Teng Jian Khoo requested to merge khoo/athena:FTF-dedx-fix into 23.0

Incorrect GeV to MeV conversion broke the dE/dx trigger. The counts went to 0 in the LLP grid test on 07-25 due to !64436 (merged).

Correcting this reverses the following count changes:

2711,2716c2711,2716
< HLT_xe80_tcpufit_dedxtrk25_medium_L1XE50                    8
< HLT_xe80_tcpufit_dedxtrk25_medium_L1XE55                    8
< HLT_xe80_tcpufit_dedxtrk25_medium_L1jXE100                  8
< HLT_xe80_tcpufit_dedxtrk50_medium_L1XE50                    7
< HLT_xe80_tcpufit_dedxtrk50_medium_L1XE55                    7
< HLT_xe80_tcpufit_dedxtrk50_medium_L1jXE100                  7
---
> HLT_xe80_tcpufit_dedxtrk25_medium_L1XE50                    0
> HLT_xe80_tcpufit_dedxtrk25_medium_L1XE55                    0
> HLT_xe80_tcpufit_dedxtrk25_medium_L1jXE100                  0
> HLT_xe80_tcpufit_dedxtrk50_medium_L1XE50                    0
> HLT_xe80_tcpufit_dedxtrk50_medium_L1XE55                    0
> HLT_xe80_tcpufit_dedxtrk50_medium_L1jXE100                  0

The other fix is for a simultaneous bug in the hit-based displaced vertex trigger, which raised the track pt cut from 0.5 GeV to 3 GeV, with the intended result:

< HLT_hitdvjet200_medium_L1XE50                               9
< HLT_hitdvjet200_medium_L1XE55                               9
< HLT_hitdvjet200_medium_L1jXE100                             7
< HLT_hitdvjet260_medium_L1J100                               3
< HLT_hitdvjet260_medium_L1jJ160                              3
---
> HLT_hitdvjet200_medium_L1XE50                              12
> HLT_hitdvjet200_medium_L1XE55                              11
> HLT_hitdvjet200_medium_L1jXE100                            11
> HLT_hitdvjet260_medium_L1J100                               5
> HLT_hitdvjet260_medium_L1jJ160                              5
1205,1208c1205,1208
< HLT_j180_hitdvjet200_medium_L1XE50                          9
< HLT_j180_hitdvjet200_medium_L1XE55                          9
< HLT_j180_hitdvjet260_medium_L1J100                          3
< HLT_j180_hitdvjet260_medium_L1jJ160                         3
---
> HLT_j180_hitdvjet200_medium_L1XE50                         12
> HLT_j180_hitdvjet200_medium_L1XE55                         11
> HLT_j180_hitdvjet260_medium_L1J100                          5
> HLT_j180_hitdvjet260_medium_L1jJ160                         5

I incidentally fixed an erroneous unordered_map declaration, which isn't needed here.

Edited by Teng Jian Khoo

Merge request reports