Add three new lines to select charged kaon decays in HLT1 for Run 3
This MR includes a 3-body SV maker (as compared to the current 2+1 combinations) and a set of lines to select three-body charged kaon decays in HLT1. For now only
Hlt2RD_KpToPiMuMu
, Hlt2RD_KpToPiPiPi
and Hlt2RD_KpToPiEE
. In addition, the All lines have been tested locally on expected 2024 minimum bias bandwidth division samples with the sequence hlt1_pp_forward_then_matching_1200KHz
yielding the following rates:
Hlt1Kplus2PiEE: 9/200000, ( 1.35 +/- 0.45) kHz
Hlt1Kplus2PiMuMu: 6/200000, ( 0.90 +/- 0.37) kHz
Hlt1Kplus2PiPiPi: 2/200000, ( 0.30 +/- 0.21) kHz
Inclusive: 9909/200000, ( 1486.35 +/- 14.56) kHz
Without the new lines for comparison:
Inclusive: 9895/200000, ( 1484.25 +/- 14.55) kHz
So the lines add about 2 kHz.
The throughput check yields
Device Throughput (kHz) Reference Throughput (kHz) Speedup % change Status
-------------------------- ------------------ ---------------------------- --------- ---------- --------
NVIDIA RTX A5000 75.13 76.14 0.99x -1.34% OK
AMD EPYC 7502 32-Core 8.03 8.42 0.95x -4.61% OK
NVIDIA GeForce RTX 2080 Ti 53.72 56.09 0.96x -4.23% OK
NVIDIA GeForce RTX 3090 92.44 93.64 0.99x -1.28% OK
Device-averaged speedup: 0.97x
% change: -2.87%
status: DECREASED
with a few % slowdown on the CPU and 2080 NVIDIA GeForce RTX 2080 Ti card and ~ 1% slowdown in the rest. This decrease is mainly due to the 3-body SVs combiner which this MR adds.
Todo:
-
Test using 2024 expected minbias and signal MC -
Add LFV decay combinations (this could be postponed to a second stage) -
Adapt to the new monitoring machinery from @ahennequ once it is merged (see !1483 (merged))
Uses the new 2+1 body combiner machinery from @mstahl and @thboettc
fyi @fdettori @acontu @fdordei
Reviewer assignment
Merge request reports
Activity
requested review from @mstahl
added RTA label
- Resolved by Thomas Boettcher
- Resolved by Thomas Boettcher
- Resolved by Thomas Boettcher
mentioned in merge request Moore!3429 (merged)
marked the checklist item Adapt to the new monitoring machinery from @ahennequ once it is merged (see !1483 (merged)) as completed
marked the checklist item Adapt to the new monitoring machinery from @ahennequ once it is merged (see !1483 (merged)) as incomplete
assigned to @thboettc
added 443 commits
-
af1e30ce...df8e0fdc - 440 commits from branch
2024-patches
- aa917ff0 - First commit of lines to select charged kaon decays in HLT1 for Run 3
- 6e147a15 - Fixed formatting
- d5642898 - Flatten 2+1 SVs to make 3-body SVs.
Toggle commit list-
af1e30ce...df8e0fdc - 440 commits from branch
added 93 commits
- 73ba522c...76813c15 - 83 earlier commits
- c9373340 - Add J/psi TagProbe line for SMOG2 to improve MuonID efficiency studies
- b5200ee4 - Merge branch 'samarian_TagProbeSMOGJpsi' into 'master'
- 8228cdb1 - Merge branch 'master' into dmendoza_charged_kaon_decays
- a4cbe59d - Release v5r1
- 6d462c19 - Merge branch 'release_v5r1' into 'master'
- ab541d70 - Merge branch 'dmendoza_charged_kaon_decays' into vava_charged_kaon_decays
- 993d11c8 - Fix refs for allen_event_loop
- f1f2d9e9 - Merge branch 'allen_event_loop-fix-refs' into 'master'
- c009bffe - Updating with master and solving conflicts
- 6bcad84e - Merge branch 'master' into vava_charged_kaon_decays
Toggle commit listadded 1 commit
- adb274ef - Fixing bugs + opening line amap to study drop in eff