Skip to content
Snippets Groups Projects

TrigCost: Move end event of TrigCostSvc to dedicated algorithm

Merged Aleksandra Poreba requested to merge aporeba/athena:cost-in-end-of-event into 22.0
All threads resolved!

Move the end event of TrigCostSvc from DecisionSummaryMaker to the dedicated algorithm executed a bit later. The configuration in the sequence (for athenaHLT):

            HLTEndSeq [Seq]  [Concurrent]  [OR] 
              TriggerSummaryFinal [Alg]  [n= 0]
              TrigSignatureMoni [Alg]  [n= 0]
              HLTFinalizeSeq [Seq]  [Sequential]  [Prompt] 
                DecisionSummaryMakerAlg [Alg]  [n= 0]
                EndOfEventROIConfirmerAlg [Alg]  [n= 0]
                acceptedEventTopSeq [Seq]  [Concurrent]  [OR]  [PASS] 
                  acceptedEventSeq_larnoiseburst_L1All [Seq]  [Sequential]  [Prompt] 
                  acceptedEventSeq_metcalo_L1All [Seq]  [Sequential]  [Prompt] 
                  acceptedEventSeq_mettrk_L1All [Seq]  [Sequential]  [Prompt] 
                    EndOfEventFilterAlg_mettrk_L1All [Alg]  [n= 0]
                    acceptedevtsmettrkEndOfEventRecoSequence [Seq]  [Concurrent]  [OR] 
                TrigCostFinalizeAlg [Alg]  [n= 0]
                EDMCreatorAlg [Alg]  [n= 0]
                TrigNavSlimmingMTAlg_Online [Alg]  [n= 0]

Example results with some of the algorithms that run after DecisionSummaryMaker: https://atlas-trig-cost-dev.web.cern.ch/?dir=aporeba&type=&tag=late-costmon-oldjo&run=431885&range=LumiBlock_00500&level=HLT&summary=Algorithm&query=EndOfEventFilterAlg_larnoiseburst_L1All

Related to ATR-26011

Tagging @tamartin

Merge request reports

Pipeline #4654126 passed

Pipeline passed for a5b88d5f on aporeba:cost-in-end-of-event

Approval is optional

Merged by Marija Vranjes MilosavljevicMarija Vranjes Milosavljevic 2 years ago (Oct 24, 2022 8:43am UTC)

Merge details

  • Changes merged into 22.0 with a5454088 (commits were squashed).
  • 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
  • added 1 commit

    • a5b88d5f - Bring back the recordable cost collection

    Compare with previous version

  • Aleksandra Poreba resolved all threads

    resolved all threads

  • :pencil: INFO: fewer projects are affected by code changes: Athena

  • This merge request affects 4 packages:

    • Trigger/TrigMonitoring/TrigCostMonitor
    • Trigger/TrigSteer/TrigOutputHandling
    • Trigger/TriggerCommon/TriggerJobOpts
    • Trigger/TriggerCommon/TriggerMenuMT

    Affected files list will not be printed in this case

    Adding @tamartin ,@aporeba ,@shanisch ,@martindl ,@sutt ,@bernius ,@rbielski ,@lidiaz as watchers

  • :white_check_mark: CI Result SUCCESS (hash a5b88d5f)

    Athena DetCommon
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    tests :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 60683]

  • Approving, L1

  • mentioned in commit a5454088

  • Please register or sign in to reply
    Loading