Skip to content
Snippets Groups Projects

added express stream to egamma, muon, and tau chains for the express stream (from the draft list). ATR-24283 , ATR-24367

Merged Gianantonio Pezzullo requested to merge gipezzul/athena:exprDev into master

I used the list of chains in the google-spreadsheet linked in ATR-24367 for adding the express stream to the chains specified for the following signatures:

  • egamma
  • muon
  • Tau

I have a few questions:

  1. for those chains that have NO monGroups specified yet, which monGroup should I specify @shanisch ? If I had to modify a chain listed also in ATR-242283, I added all the monGroups I had to, but in case no monGroup was specified, I added the "t0" one. Is that enough/correct?
  2. for the Tau case, a this link it's unclear which chains I should use. @ademaria can you please clarify?
  3. for the Muon signature, one of the chains listed for the express stream "HLT_mu13_mu13_idperf_zmumu" is missing from the current Menu files. Which one should I use @nakahama @cdiez ?

@shanisch with your answer to question (1), I can move forward and add the express stream (+ the monGroup) to the other chains I collected from the other signatures responsible.

Thanks for the help!

Edited by Gianantonio Pezzullo

Merge request reports

Pipeline #3294303 passed

Pipeline passed for 1a0ad39e on gipezzul:exprDev

Merged by Vakhtang TsulaiaVakhtang Tsulaia 3 years ago (Nov 29, 2021 7:40pm UTC)

Merge details

  • Changes merged into master with b23402e8 (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • as discussed in today's meeting, please remove the monGroups for chains for which they weren't requested yet even if these chains were requested to be added to the express stream. this affects the following chains:

    • HLT_2g20_tight_icaloloose_L12eEM15M
    • HLT_g35_medium_g25_medium_L12eEM20L
    • HLT_e26_lhtight_e15_etcut_idperf_probe_50invmAB130_L1eEM22M
    • HLT_e26_lhtight_e15_etcut_probe_50invmAB130_L1EM22VHI
    • HLT_e9_lhtight_e4_etcut_1invmAB5_L1JPSI-1M5-EM7
    • HLT_e14_lhtight_noringer_e4_etcut_1invmAB5_L1JPSI-1M5-EM12
    • HLT_tau160_mediumRNN_tracktwoMVABDT_L1eTAU100
    • HLT_tau180_mediumRNN_tracktwoLLP_L1eTAU100
    • HLT_2mu4_bJpsimumu_L12MU3V
    • HLT_e26_lhtight_e15_etcut_L1EM22VHI
    • HLT_e26_idperf_tight_L1EM22VHI
    • HLT_tau0_ptonly_L1TAU60
    • HLT_mu6_idperf_L1MU5VF
    • HLT_mu24_idperf_L1MU14FCH
    • HLT_2e24_lhvloose_L12EM20VH
    • HLT_tau35_mediumRNN_tracktwoMVABDT_tau25_mediumRNN_tracktwoMVABDT_03dRAB30_L1DR
    • HLT_mu20_ivarloose_tau20_mediumRNN_tracktwoMVABDT_03dRAB_L1MU14FCH
    • HLT_e24_lhmedium_ivarloose_tau20_mediumRNN_tracktwoMVABDT_03dRAB_L1EM22VHI
  • added 4 commits

    • 0a52f1d2 - removed monGroups to the chains in the express stream that have no monGroup specified yet
    • 12984dac - clean up in the Tau express-chains
    • c421f7e3 - moved some chains from LS2 to Physics_pp Menu file
    • 123ef04f - fixed group assignment

    Compare with previous version

  • Hi @shanisch ,

    Thanks for checking. I implemented the clean up you requested.

    Cheers,

    Giani

  • added 1 commit

    • d20a74fa - fixed monGroup for hlt_e26_lhtight_ivarloose_L1M22VHI

    Compare with previous version

  • Gianantonio Pezzullo resolved all threads

    resolved all threads

  • Gianantonio Pezzullo marked this merge request as ready

    marked this merge request as ready

  • :warning: WARNING: big files (>100K) are found in the changeset

    :pencil: 116K in file Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/Physics_pp_run3_v1.py

    :pencil: 128K in file Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/LS2_v1.py

  • This merge request affects 1 package:

    • Trigger/TriggerCommon/TriggerMenuMT

    This merge request affects 2 files:

    • Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/LS2_v1.py
    • Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/Physics_pp_run3_v1.py

    Adding @sutt ,@dzanzi ,@iriu ,@carquin ,@ademaria ,@bernius ,@hrussell ,@malconad as watchers

  • Gianantonio Pezzullo marked this merge request as draft

    marked this merge request as draft

  • added 1 commit

    • f22e5604 - fixed typos + added hlt_tau25_mediumRNN_tracktwoMVABDT_L1TAU12IM

    Compare with previous version

  • Gianantonio Pezzullo marked this merge request as ready

    marked this merge request as ready

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