Changeset 6f004b5 in git

Timestamp:

Jul 13, 2020, 5:36:52 PM (4 years ago)

Author:

GitHub <noreply@…>

Branches:

master

Children:

f9aeea6

Parents:

c18dca6

git-author:

jared-burleson <55971332+jared-burleson@…> (07/13/20 17:36:52)

git-committer:

GitHub <noreply@…> (07/13/20 17:36:52)

Message:

Bug-fix FastJetFinder.cc in fComputeTrimming

Fixed a bug with the Trimming Method where individual subjets would be constructed with only 1 constituent, now allows for subjets with multiple constituents.

File:

• modules/FastJetFinder.cc (modified) (1 diff)

modules/FastJetFinder.cc

@@ -490 +490 @@
       fastjet::PseudoJet trimmed_jet = trimmer(*itOutputList);
 
-      trimmed_jet = join(trimmed_jet.constituents());
-
+      //trimmed_jet = join(trimmed_jet.constituents());
+      
+      //The previous line generates a trimmed_jet where each piece (subjet) has a single constituent only
+      //This isn't necessarily correct, a subjet could have multiple constituents, according to defining trimming paper
+      //Removing this line retains the trimmed_jet P4 but also allows subjets to be formed with multiple constituents
+      //based on the R-parameter requirements, and is consistent with the ATLAS/ATHENA code from their github respository
+      //Jared Burleson, Undergraduate Student at Southern Methodist University
+      
       candidate->TrimmedP4[0].SetPtEtaPhiM(trimmed_jet.pt(), trimmed_jet.eta(), trimmed_jet.phi(), trimmed_jet.m());