Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • Allen Allen
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 183
    • Issues 183
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Jira
    • Jira
  • Merge requests 73
    • Merge requests 73
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • AllenAllen
  • Merge requests
  • !1112

Draft: Remove GEC from default HLT1 sequence

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Ahmed Abdelmotteleb requested to merge ahabdelm-remove-gec-from-hlt1 into master Jan 30, 2023
  • Overview 10
  • Commits 13
  • Pipelines 7
  • Changes 22

Closes #395

Renders !664 (closed) and !784 (closed) obsolete

@dovombru

Want to remove Global Event Cuts (GEC) from the default HLT1 sequence. All HLT1 would then run without these cuts being implemented at the beginning of data processing.

Also want to study the impact on the selection efficiency for Z\rightarrow\mu\mu, W\rightarrow\mu\nu, and other processes impacted by the GEC

Changes so far:

  • Set the default hlt1 node to NOT include GEC (EnableGEC=False in HLT1.py)
  • Remove GEC from all sequence files in configuration/python/AllenSequences

- Added a new function called make_composite_node to replace the currrently used function make_composite_node_with_gec which is featured in a couple of sequence files (is there a better way to do this?) -->Unnecessary

Next plans:

  • Study the impact on the selection efficiency for Z\rightarrow\mu\mu
  • Study the impact on the selection efficiency for W\rightarrow\mu\nu
  • Confirm that the impact on the throughput is reasonable

Throughput:

Latest throughput values obtained from the "AllenPR throughput" mattermost channel:

TL;DR: all throughputs are fine, apart from the one from hlt1_pp_no_ut which is slightly worse than before:

  • hlt1_pp_default hlt1_pp_default

  • hlt1_pp_no_gec: hlt1_pp_no_gec

  • hlt1_pp_matching: hlt1_pp_matching

  • hlt1_pp_matching_no_gec --> script has some changes from master: hlt1_pp_matching_no_gec

  • hlt1_pp_no_ut: hlt1_pp_no_ut


Efficiencies:

Studied the efficiencies for Z/W decays using HltEfficiencyChecker and 2 yaml files: hlt1_z_eff.yaml hlt1_w_eff.yaml

Can just run the command: MooreAnalysis/run MooreAnalysis/HltEfficiencyChecker/scripts/hlt_eff_checker.py hlt1_w_eff.yaml from the stack home directory

TL;DR: both W and Z efficiencies increase without GEC as expected (0.914 --> 0.941 and 0.935 --> 0.977 respectively)

W before (master branch): w_eff_check_master_branch.txt

Integrated HLT efficiencies for the lines with denominator: CanRecoChildren
------------------------------------------------------------------------------------
Line:	 Hlt1SingleHighPtMuonDecision         	 Efficiency:	 0.914 +/- 0.009
Line:	 W_Hlt1SingleHighPtMuonDecisionTrueSim	 Efficiency:	 0.914 +/- 0.009
------------------------------------------------------------------------------------
Finished printing integrated HLT efficiencies for denominator: CanRecoChildren
------------------------------------------------------------------------------------

W after (this branch): w_eff_check_new_branch.txt

Integrated HLT efficiencies for the lines with denominator: CanRecoChildren
------------------------------------------------------------------------------------
Line:	 Hlt1SingleHighPtMuonDecision         	 Efficiency:	 0.941 +/- 0.008
Line:	 W_Hlt1SingleHighPtMuonDecisionTrueSim	 Efficiency:	 0.940 +/- 0.008
------------------------------------------------------------------------------------
Finished printing integrated HLT efficiencies for denominator: CanRecoChildren
------------------------------------------------------------------------------------

Z before (master branch): z_eff_check_master_branch.txt

Integrated HLT efficiencies for the lines with denominator: CanRecoChildren
------------------------------------------------------------------------------------
Line:	 Hlt1SingleHighPtMuonDecision          	 Efficiency:	 0.935 +/- 0.015
Line:	 Z0_Hlt1SingleHighPtMuonDecisionTrueSim	 Efficiency:	 0.935 +/- 0.015
------------------------------------------------------------------------------------
Finished printing integrated HLT efficiencies for denominator: CanRecoChildren
------------------------------------------------------------------------------------

Z after (this branch): z_eff_check_new_branch.txt

Integrated HLT efficiencies for the lines with denominator: CanRecoChildren
------------------------------------------------------------------------------------
Line:	 Hlt1SingleHighPtMuonDecision          	 Efficiency:	 0.977 +/- 0.009
Line:	 Z0_Hlt1SingleHighPtMuonDecisionTrueSim	 Efficiency:	 0.977 +/- 0.009
------------------------------------------------------------------------------------
Finished printing integrated HLT efficiencies for denominator: CanRecoChildren
------------------------------------------------------------------------------------

I also checked with the default HLTEfficiencyChecker example: MooreAnalysis/run MooreAnalysis/HltEfficiencyChecker/scripts/hlt_eff_checker.py MooreAnalysis/HltEfficiencyChecker/options/hlt1_eff_example.yaml and confirmed that these lines were not affected (as expected): default_eff_check_master_branch.txtdefault_eff_check_new_branch.txt

Efficiencies plots:

Got some plots using HltEfficiencyChecker:

Efficiencies__Hlt1SingleHighPtMuonDecision__CanRecoChildren__PT:

  • W before: Efficiencies__Hlt1SingleHighPtMuonDecision__CanRecoChildren__PT
  • W after: Efficiencies__Hlt1SingleHighPtMuonDecision__CanRecoChildren__PT
  • Z before: Efficiencies__Hlt1SingleHighPtMuonDecision__CanRecoChildren__PT
  • Z after: Efficiencies__Hlt1SingleHighPtMuonDecision__CanRecoChildren__PT
Edited Mar 19, 2023 by Ahmed Abdelmotteleb
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: ahabdelm-remove-gec-from-hlt1