@@ -213,9 +213,10 @@ class McParticleHistManager
213213 if (mPlotOrigins ) {
214214 // NOTE: kNoMcParticle and kMissidentified are kept here only for enum/histogram
215215 // symmetry with TrackHist — they describe reco-vs-truth mismatches that cannot
216- // occur for a pure MC-truth particle, so these bins are expected to stay empty
217- // unless the producer assigns those origin codes for some other reason
218- // (e.g. pileup-associated particles tagged as kFromWrongCollision).
216+ // occur for a pure MC-truth particle, so these bins are expected to stay empty.
217+ // kFromWrongCollision is not handled here at all: mcBuilder.h's getOrigin() never
218+ // assigns it (only kPhysicalPrimary/kFromSecondaryDecay/kFromMaterial), so it would
219+ // be dead code the same way it was in the per-candidate hist managers.
219220 mHistogramRegistry ->fill (HIST (prefix) + HIST (McDir) + HIST (getHistName (kOrigin , HistTable)), mcParticle.origin ());
220221 switch (static_cast <modes::McOrigin>(mcParticle.origin ())) {
221222 case modes::McOrigin::kNoMcParticle :
@@ -224,9 +225,6 @@ class McParticleHistManager
224225 case modes::McOrigin::kPhysicalPrimary :
225226 mHistogramRegistry ->fill (HIST (prefix) + HIST (McDir) + HIST (getHistName (kPrimary , HistTable)), mcParticle.pt ());
226227 break ;
227- case modes::McOrigin::kFromWrongCollision :
228- mHistogramRegistry ->fill (HIST (prefix) + HIST (McDir) + HIST (getHistName (kFromWrongCollision , HistTable)), mcParticle.pt ());
229- break ;
230228 case modes::McOrigin::kFromMaterial :
231229 mHistogramRegistry ->fill (HIST (prefix) + HIST (McDir) + HIST (getHistName (kFromMaterial , HistTable)), mcParticle.pt ());
232230 break ;
0 commit comments