search

UCATLAS / xAODAnaHelpers

ATLAS Run 2 and Run 3 analysis framework for AnalysisTop and AnalysisBase for proton-smashing physics
https://ucatlas.github.io/xAODAnaHelpers/
Apache License 2.0
42 stars 127 forks source link

Issue implementing trigger decisions with TreeAlgo #1662

Closed max-cohen54 closed 1 year ago

max-cohen54 commented 1 year ago

Hello,

I'm using xAODAnaHelpers r22v1.0.1, and I'm trying to add the trigger decisions of events to a tree with TreeAlgo. Ideally, I would write down a list of triggers, and it would include whether or not each of those triggers were passed for each event. If the only possible implementation is to include the trigger decisions of all triggers (rather than just a specified list), that would in principle be okay as well.

Based on my understanding from reading TreeAlgo and HelpTreeBase (and TriggerInfoSwitch in HelperClasses), I should be able to include m_trigDetailStr in my config file, which can take values such as "basic", "passTriggers", etc. However, when I include this in the cofig file, additional branches are made in the tree (passL1, passHLT, passedTriggers, disabledTriggers), but these branches are empty.

Is there something I am doing wrong with this implementation? Thanks!

gist: https://gist.github.com/max-cohen54/b70e107f1c08ca1256340c9555c318e7

max-cohen54 commented 1 year ago

Solved!

kratsg commented 1 year ago

you might want to describe how you fixed it...

max-cohen54 commented 1 year ago

It turns out I wasn't saving the trigger decisions in BasicEventSelection, so they were never making it into my tree. I had to use m_triggerSelection to list the triggers I was interested in, and also m_storeTriggerDecisions for this information to be stored. Then, using m_trigDetailStr with TreeAlgo saved the trigger decisions in the tree!