Skip to content
Snippets Groups Projects

Update SMOG2 KS trackeff lines in Turcal

Merged Oscar Boente Garcia requested to merge updates-ks-smog2-tracking into master

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

image

image

Edited by Tommaso Fulghesu

Merge request reports

Merge request pipeline #11696298 passed with warnings

Merge request pipeline passed with warnings for 6cfc5934

Merged by Miroslav SaurMiroslav Saur 5 days ago (Apr 11, 2025 5:37am UTC)

Loading

Pipeline #11718492 failed

Pipeline failed for 77006c90 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Oscar Boente Garcia marked the checklist item Add tis filter for HLT1 unbiasing as completed

    marked the checklist item Add tis filter for HLT1 unbiasing as completed

  • Oscar Boente Garcia marked the checklist item Add selective persistency of matched track as completed

    marked the checklist item Add selective persistency of matched track as completed

  • Oscar Boente Garcia marked the checklist item Remove RAW banks (not needed with selective persistency) as completed

    marked the checklist item Remove RAW banks (not needed with selective persistency) as completed

  • Oscar Boente Garcia changed the description

    changed the description

  • Oscar Boente Garcia marked the checklist item Readjust selection to reduce background and improve kinematic converage, update prescales as completed

    marked the checklist item Readjust selection to reduce background and improve kinematic converage, update prescales as completed

  • added 18 commits

    Compare with previous version

  • added 4 commits

    • 36278b2c - Add Tos filter, and selective persistency
    • b8236bbe - Change line names to Turbo convention and fix typos
    • c6915e8f - Unify SMOG2 and pp Ks VeloLong builders
    • d7778b22 - Adjust P requirements for SMOG2

    Compare with previous version

  • added 1 commit

    • bcf4943b - fix bug in fromSMOG2 argument definition

    Compare with previous version

  • Oscar Boente Garcia marked the checklist item Unifiy code with KSVeloLong (with pp lines, following Sasha and Laurent suggestion) as completed

    marked the checklist item Unifiy code with KSVeloLong (with pp lines, following Sasha and Laurent suggestion) as completed

  • Oscar Boente Garcia marked the checklist item Remove Long track containers (keep only MatchedLongPions and MatchedUpstreamPions) as completed

    marked the checklist item Remove Long track containers (keep only MatchedLongPions and MatchedUpstreamPions) as completed

  • Oscar Boente Garcia marked the checklist item Test new Bandwidht with SMOG2 data, compare with previous one (expect a reduction thanks to moving to Turbo strategy) as completed

    marked the checklist item Test new Bandwidht with SMOG2 data, compare with previous one (expect a reduction thanks to moving to Turbo strategy) as completed

  • Oscar Boente Garcia marked the checklist item Check if I need to change the line names, or if further changes are needed in hlt2/sprucing sequence (do lines need Turbo in the name now?) as completed

    marked the checklist item Check if I need to change the line names, or if further changes are needed in hlt2/sprucing sequence (do lines need Turbo in the name now?) as completed

  • Oscar Boente Garcia changed the description

    changed the description

  • Sascha Stahl
  • 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

    Compare with previous version

  • Oscar Boente Garcia marked this merge request as ready

    marked this merge request as ready

  • Oscar Boente Garcia changed the description

    changed the description

  • added 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

    Compare with previous version

  • Oscar Boente Garcia changed the description

    changed the description

  • added 1 commit

    Compare with previous version

  • added 1 commit

    • e13c4528 - Rename functions smog2 ks lines

    Compare with previous version

  • Sascha Stahl approved this merge request

    approved this merge request

  • added 1 commit

    Compare with previous version

  • Oscar Boente Garcia reset approvals from @sstahl by pushing to the branch

    reset approvals from @sstahl by pushing to the branch

  • added 1 commit

    Compare with previous version

  • added IFT label

  • Edited by Software for LHCb
  • added 1 commit

    • e2af44dc - register smog2 ks track lines in all_lines

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading