From 2305fbf42b350ce8957d00c34d11af11f283cbc5 Mon Sep 17 00:00:00 2001 From: Maria Mazza Date: Tue, 18 Aug 2026 16:55:27 -0400 Subject: [PATCH] FastJetAlg: return the jets in the jetOut slot The MultiTransformer declares its two outputs jets-first: {KeyValues("jetOut", {"JetOut"}), KeyValues("recParticleOut", {"Constituents"})} but operator() returned them the other way round, so the collection written under jetOut received `outputCollection` (the subset of input particles that were clustered, flagged with setSubsetCollection(true)) and the collection written under recParticleOut received `jetCollection` (the jets). The declared order is the intended one -- the default names are "JetOut" for slot 0 and "Constituents" for slot 1 -- so fix the return, not the declaration. --- k4Reco/FastJet/src/FastJetAlg.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/k4Reco/FastJet/src/FastJetAlg.cpp b/k4Reco/FastJet/src/FastJetAlg.cpp index 7ce6c7e..c820f5c 100644 --- a/k4Reco/FastJet/src/FastJetAlg.cpp +++ b/k4Reco/FastJet/src/FastJetAlg.cpp @@ -348,5 +348,5 @@ FastJetAlg::operator()(const edm4hep::ReconstructedParticleCollection& inputColl } } - return std::make_tuple(std::move(outputCollection), std::move(jetCollection)); + return std::make_tuple(std::move(jetCollection), std::move(outputCollection)); } \ No newline at end of file