Open saswatinandan opened 1 year ago
@ktht , did you miss this pull request by any chance or I made some major mistakes here?
Please test the code. I sense that there will be segfaults.
I guess there is no option to test it, as the corresponding channel is missing now, I belive
But you can change the config minimally such that the relevant piece of code will be executed.
I tried to check this but it is not possible to do by minimal changes as there is error not coming from trigger but before that. I don't see any such thing in cfg file https://github.com/HEP-KBFI/TallinnNtupleProducer/blob/main/EvtWeightTools/src/HadTauFakeRateInterface.cc#L86 I need one old cfg file to see how it works.
But you can change the config minimally such that the relevant piece of code will be executed.
checked on 4tau samples for 1k events, didn't encounter any error
This is related to the issue. I guess we can simply check the filter bits of reco object to decide whether they are matched with trigger objects and if yes, consider only those reco objects to calculate sf. If it is fine, I'll check for 1l_3tau trigger also, currently implemented only for 4tau trigger case