Skip to content

BoostedJetTaggers overhaul

Jason Robert Veatch requested to merge jveatch/athena:21.2-BJT into 21.2

This is a major overhaul of the BoostedJetTaggers package. The goal is to significantly clean up and simplify the many classes that have been developed independently.

A lot of redundant functionality is moved JSSTaggerBase, a lot of code is rewritten/cleaned up to be more readable and easier to maintain, variable names are changed to fit the ATLAS coding guidelines, comments are cleaned up and implemented in doxygen style. Furthermore, the static variables used to count the number of warnings printed are replaced with std::atomic for thread safety. Most of the changes are limited only to the classes that will be ported to master.

TODO: Add ability to use new UFO taggers

Local tests have shown no difference in outputs beyond floating point precision.

Tagging @tnobe, @farooque, @cdelitzs, @mswiatlo, @wbalunas, and @sawyer for information and comments. Also tagging @schae for feedback on the changes to the QG tagging tools.

Edited by Jason Robert Veatch

Merge request reports