Update SMOG2 KS trackeff lines in Turcal
This MR aims to add some updates and improvements to the Ks->pipi tracking efficiency lines using the VELO-Long method with Ks candidates coming from SMOG2. They mostly follow what is already in the equivalent lines for pp (https://gitlab.cern.ch/lhcb/Moore/-/blob/master/Hlt/Hlt2Conf/python/Hlt2Conf/lines/trackeff/KSVeloLong.py?ref_type=heads).
To do:
-
Add tis filter for HLT1 unbiasing -
Add selective persistency of matched track -
Remove RAW banks (not needed with selective persistency) -
Readjust selection to reduce background and improve kinematic converage, update prescales -
Unifiy code with KSVeloLong (with pp lines, following Sasha and Laurent suggestion) -
Remove Long track containers (keep only MatchedLongPions and MatchedUpstreamPions) -
Test new Bandwidht with SMOG2 data, compare with previous one (expect a reduction thanks to moving to Turbo strategy) -
Check if I need to change the line names, or if further changes are needed in hlt2/sprucing sequence
Local Bandwidth tests results (on a representative pp+pAr sample from Block 1, the same as in !4500 (merged) ). Note that this data is from block 1 (mu pp 4.4), so the SMOG2 bandwidth is representative while the bandwidth of pp lines should be below that one in mu=5.3 conditions.
Before any changes (baseline, as in master)
Line | Total Retention (%) | Rate (kHz) | Exclusive Retention(%) | Exclusive Rate (kHz) | Avg Total Event Size (kB) | Total Bandwidth (GB/s) | Avg DstData Size (kB) | DstData Bandwidth (GB/s) | |
---|---|---|---|---|---|---|---|---|---|
6 | SpruceTurCalTrackEff_SMOG2KshortVeloLong_HighPTDecision | 1.19 | 0.108 | 1.19 | 0.108 | 44.1 | 0.00476 | 1.84 | 0.000199 |
9 | SpruceTurCalTrackEff_SMOG2KshortVeloLong_LowPTDecision | 0.927 | 0.084 | 0.927 | 0.084 | 40.4 | 0.00339 | 2.42 | 0.000203 |
11 | SpruceTurCalTrackEff_SMOG2KshortVeloLong_VeryHighPTDecision | 0.662 | 0.06 | 0.662 | 0.06 | 51.9 | 0.00312 | 1.8 | 0.000108 |
Line | Total Retention (%) | Rate (kHz) | Exclusive Retention(%) | Exclusive Rate (kHz) | Avg Total Event Size (kB) | Total Bandwidth (GB/s) | Avg DstData Size (kB) | DstData Bandwidth (GB/s) | |
---|---|---|---|---|---|---|---|---|---|
0 | Hlt2TurboVelo2Long_KshortHardDecision | 0.03 | 0.36 | 0.03 | 0.36 | 52.4 | 0.0189 | 51.9 | 0.0187 |
1 | Hlt2TurboVelo2Long_KshortVHardDecision | 0.016 | 0.192 | 0.016 | 0.192 | 64.8 | 0.0124 | 64.3 | 0.0123 |
2 | Hlt2TurboVelo2Long_KshortDecision | 0.014 | 0.168 | 0.014 | 0.168 | 66 | 0.0111 | 65.6 | 0.011 |
3 | Hlt2TurboVelo2Long_KshortSoftDecision | 0.012 | 0.144 | 0.012 | 0.144 | 70.8 | 0.0102 | 70.3 | 0.0101 |
4 | Hlt2TurboVelo2Long_KshortVSoftDecision | 0.007 | 0.084 | 0.007 | 0.084 | 61.2 | 0.00514 | 60.8 | 0.00511 |
(note that here Long and Upstream containers were persisted, as changes from !4407 (merged) had not been merged by then)
As with the proposed changes
Merge request reports
Activity
changed milestone to %RTA 2025 Data taking
assigned to @oboenteg
added RTA label
- Resolved by Tommaso Fulghesu
Is there a need to have these lines in a different module? Then we could avoid copy paste.
- Resolved by Oscar Boente Garcia
added 18 commits
-
d40d0d8b...7cfe4bb4 - 16 commits from branch
master
- 9a0b1d01 - Add Tos filter, and selective persistency
- 7b2a6152 - Change line names to Turbo convention and fix typos
-
d40d0d8b...7cfe4bb4 - 16 commits from branch
- Resolved by Sascha Stahl
- Resolved by Oscar Boente Garcia
- Resolved by Oscar Boente Garcia
- Resolved by Oscar Boente Garcia
added 6 commits
- 1dc118c0 - Add Tos filter, and selective persistency
- c16dec22 - Change line names to Turbo convention and fix typos
- ce64bf49 - Unify SMOG2 and pp Ks VeloLong builders
- 32a9b3ca - Adjust P requirements for SMOG2
- 24cac165 - fix bug in fromSMOG2 argument definition
- 78cc2ad3 - Retune SMOG2 cuts and remove upfront_reco
Toggle commit listadded 7 commits
- 63670b24 - Add Tos filter, and selective persistency
- ecfc8254 - Change line names to Turbo convention and fix typos
- 8292fefb - Unify SMOG2 and pp Ks VeloLong builders
- 2a251bb1 - Adjust P requirements for SMOG2
- bb02ab62 - fix bug in fromSMOG2 argument definition
- 9db368a6 - Retune SMOG2 cuts and remove upfront_reco
- 6b347d3f - Remove code to run without pv
Toggle commit listassigned to @tfulghes
- Resolved by Tommaso Fulghesu
- Resolved by Tommaso Fulghesu
- Resolved by Tommaso Fulghesu
Hi @oboenteg, I left a couple of trivial comments, but LGTM! If it is ready, we can test it
reset approvals from @sstahl by pushing to the branch
added PR/Moore_hlt2_and_spruce_bandwidth_SMOG2 label
added IFT label
- Resolved by Tommaso Fulghesu
/ci-test
added ci-test-triggered label
- [2025-03-24 14:15] Validation started with lhcb-master-mr#13051
- [2025-03-25 12:08] Validation started with lhcb-master-mr#13077
- [2025-03-27 15:32] Validation started with lhcb-master-mr#13117
- [2025-03-28 10:57] Validation started with lhcb-master-mr#13132
- [2025-04-07 06:34] Validation started with lhcb-master-mr#13243
- [2025-04-07 10:39] Validation started with lhcb-master-mr#13248
- [2025-04-09 18:08] Validation started with lhcb-master-mr#13302
Edited by Software for LHCb- Resolved by Oscar Boente Garcia
- Resolved by Oscar Boente Garcia