Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 138
    • Merge requests 138
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • atlasatlas
  • athenaathena
  • Merge requests
  • !47477

[ATR-24382] Update TrigDecisionMakerMT.cxx to set setEFPassedPhysics to the same as setEFPassedRaw

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Tim Martin requested to merge tamartin-master-patch-78017 into master Oct 21, 2021
  • Overview 3
  • Commits 1
  • Pipelines 1
  • Changes 1

This lets the eventindex use EFPassedPhysics as the source of EF (R1) / HLT (R2, R3) trigger bits.

In R3 these bits are the same the pass raw bits (the TrigDefs::EnforceLogicalFlow check is not explicitly applied here, but it is implied in a two-level system, seeded from L1. The only thing which could break this is the Trigger.HLTSeeding.forceEnableAllChains flag, but this is only used for testing and is being phased out from use in testing too)

In R2, R1 there are additional complications, here the EFPassedPhysics bits are set via the TrigDecisionTool's logic. This happens currently in TrigDecisionMaker still.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: tamartin-master-patch-78017