From 1da1a69bf3d9830c3c245ea55d2d7f03dfdfaab2 Mon Sep 17 00:00:00 2001 From: Manos Vourliotis Date: Wed, 29 May 2024 16:24:09 -0700 Subject: [PATCH 1/2] Sync with runtime flags from standalone --- RecoTracker/LST/plugins/alpaka/LSTProducer.cc | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/RecoTracker/LST/plugins/alpaka/LSTProducer.cc b/RecoTracker/LST/plugins/alpaka/LSTProducer.cc index 811a78cbb3431..3a941bfe09613 100644 --- a/RecoTracker/LST/plugins/alpaka/LSTProducer.cc +++ b/RecoTracker/LST/plugins/alpaka/LSTProducer.cc @@ -31,6 +31,8 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE { consumes(config.getParameter("phase2OTHitsInput"))}, lstESToken_{esConsumes()}, verbose_(config.getParameter("verbose")), + nopLSDupClean_(config.getParameter("nopLSDupClean")), + tcpLSTriplets_(config.getParameter("tcpLSTriplets")), lstOutputToken_{produces()} {} void acquire(device::Event const& event, device::EventSetup const& setup) override { @@ -61,7 +63,9 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE { phase2OTHits.detId(), phase2OTHits.x(), phase2OTHits.y(), - phase2OTHits.z()); + phase2OTHits.z(), + nopLSDupClean_, + tcpLSTriplets_); } void produce(device::Event& event, device::EventSetup const&) override { @@ -77,6 +81,8 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE { desc.add("pixelSeedInput", edm::InputTag{"lstPixelSeedInputProducer"}); desc.add("phase2OTHitsInput", edm::InputTag{"lstPhase2OTHitsInputProducer"}); desc.add("verbose", 0); + desc.add("nopLSDupClean", 0); + desc.add("tcpLSTriplets", 0); descriptions.addWithDefaultLabel(desc); } @@ -84,7 +90,7 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE { edm::EDGetTokenT lstPixelSeedInputToken_; edm::EDGetTokenT lstPhase2OTHitsInputToken_; device::ESGetToken, TrackerRecoGeometryRecord> lstESToken_; - const int verbose_; + const int verbose_, nopLSDupClean_, tcpLSTriplets_; edm::EDPutTokenT lstOutputToken_; SDL::LST lst_; From cead1b61265a476714f683236df31ac8bb642387 Mon Sep 17 00:00:00 2001 From: Manos Vourliotis Date: Thu, 30 May 2024 06:29:38 -0700 Subject: [PATCH 2/2] Switch some parameters to booleans --- RecoTracker/LST/plugins/alpaka/LSTProducer.cc | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/RecoTracker/LST/plugins/alpaka/LSTProducer.cc b/RecoTracker/LST/plugins/alpaka/LSTProducer.cc index 3a941bfe09613..529928ec66822 100644 --- a/RecoTracker/LST/plugins/alpaka/LSTProducer.cc +++ b/RecoTracker/LST/plugins/alpaka/LSTProducer.cc @@ -30,9 +30,9 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE { lstPhase2OTHitsInputToken_{ consumes(config.getParameter("phase2OTHitsInput"))}, lstESToken_{esConsumes()}, - verbose_(config.getParameter("verbose")), - nopLSDupClean_(config.getParameter("nopLSDupClean")), - tcpLSTriplets_(config.getParameter("tcpLSTriplets")), + verbose_(config.getParameter("verbose")), + nopLSDupClean_(config.getParameter("nopLSDupClean")), + tcpLSTriplets_(config.getParameter("tcpLSTriplets")), lstOutputToken_{produces()} {} void acquire(device::Event const& event, device::EventSetup const& setup) override { @@ -80,9 +80,9 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE { edm::ParameterSetDescription desc; desc.add("pixelSeedInput", edm::InputTag{"lstPixelSeedInputProducer"}); desc.add("phase2OTHitsInput", edm::InputTag{"lstPhase2OTHitsInputProducer"}); - desc.add("verbose", 0); - desc.add("nopLSDupClean", 0); - desc.add("tcpLSTriplets", 0); + desc.add("verbose", false); + desc.add("nopLSDupClean", false); + desc.add("tcpLSTriplets", false); descriptions.addWithDefaultLabel(desc); } @@ -90,7 +90,7 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE { edm::EDGetTokenT lstPixelSeedInputToken_; edm::EDGetTokenT lstPhase2OTHitsInputToken_; device::ESGetToken, TrackerRecoGeometryRecord> lstESToken_; - const int verbose_, nopLSDupClean_, tcpLSTriplets_; + const bool verbose_, nopLSDupClean_, tcpLSTriplets_; edm::EDPutTokenT lstOutputToken_; SDL::LST lst_;