Skip to content
Snippets Groups Projects

Fix use of TrigDecisionTool in simple macro, and add example as a unittest - TEvent will now fire BeginEvent incidents

Merged Will Buttinger requested to merge will-main-patch-68034 into main

This MR will hopefully help preserve the knowledge of how to do simple tests with tools outside of a package-based environment.

The MR makes TEvent fire the BeginEvent incidents, which were previously being done by the AthenaEventLoopMgr but has since been disabled in athena. This is essential in the case of a simple eventloop job, which would otherwise require the scheduling of the AthIncFirerAlg in such loops.

Edited by Will Buttinger

Merge request reports

Loading
Loading

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

    • 05b5073c - Update macroTest.cxx - remove whitespace that snuck in for some reason when I pasted in the macro

    Compare with previous version

  • This merge request affects 3 packages:

    • Control/AthenaServices
    • PhysicsAnalysis/POOLRootAccess
    • Trigger/TrigAnalysis/TrigDecisionTool

    Affected files list will not be printed in this case

    Adding @maszyman ,@tamartin ,@gemmeren ,@ssnyder as watchers

  • added 1 commit

    • c15f2d9a - Add example of using TrigDecisionTool in a standalone python script

    Compare with previous version

  • This merge request affects 3 packages:

    • Control/AthenaServices
    • PhysicsAnalysis/POOLRootAccess
    • Trigger/TrigAnalysis/TrigDecisionTool

    Affected files list will not be printed in this case

    Adding @ssnyder ,@tamartin ,@maszyman ,@gemmeren as watchers

  • added 1 commit

    • 4edb65dc - Update macroTest.cxx - add some comments about how to access event info

    Compare with previous version

  • added 1 commit

    • 571ded0f - Update CMakeLists.txt - add python test

    Compare with previous version

  • Maciej Pawel Szymanski marked this merge request as draft

    marked this merge request as draft

  • This merge request affects 3 packages:

    • Control/AthenaServices
    • PhysicsAnalysis/POOLRootAccess
    • Trigger/TrigAnalysis/TrigDecisionTool

    Affected files list will not be printed in this case

    Adding @ssnyder ,@tamartin ,@maszyman ,@gemmeren as watchers

  • Just marking a draft to avoid numerous pipelines. Please undraft when ready.

  • yeah sorry, I'm actually all done now. Someone suggested I add a demo of python scripting as well, since python is what the kids are all into these days

    Edited by Will Buttinger
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading