Skip to content
Snippets Groups Projects

Properly setting HI flags

Merged Petr Balek requested to merge pbalek/athena:hi-fix-mc-reco into 23.0

When acquiring flags, rec.doHeavyIon was not properly translated to flags.Reco.EnableHI. This caused problem in legacy HI reconstruction as it uses wrapper for component-accumulator setup for egamma. This MR fixes that and HImode or HIPmode functions are called when appropriate, so rec.doHeavyIon and flags.Reco.EnableHI are in sync.

The flag flags.Reco.EnableHI is automatically set True if flags.Input.ProjectName has _hi in it. It worked for data (ART test has data15_hi) so the problem was not present, but not for MC (ART test has mc16_5TeV or mc21_13p6TeV).

Tagging @stapiaar

Closes ATLASRECTS-7657

Merge request reports

Pipeline #5729073 passed

Pipeline passed for 02b8a072 on pbalek:hi-fix-mc-reco

Merged by Walter LamplWalter Lampl 1 year ago (Jun 14, 2023 12:22pm UTC)

Merge details

  • Changes merged into with 19492adc.
  • Did not delete 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
Please register or sign in to reply
Loading