From 380a7ccc9d7ede98ca7f64710372df89201d4c55 Mon Sep 17 00:00:00 2001 From: Arthur Hennequin Date: Tue, 13 May 2025 09:06:09 +0200 Subject: [PATCH 1/2] Use velo clone killing on forward tracks --- Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py | 7 ++++++- Hlt/RecoConf/python/RecoConf/standalone.py | 2 +- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py b/Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py index 0b77b581051..5561ca3d421 100644 --- a/Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py +++ b/Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py @@ -31,6 +31,7 @@ from PyConf.Algorithms import ( TrackBeamLineVertexFinderSoA, UTHitClustersToPrUTHitsConverter, UTHitClustersToUTHitHandlerConverter, + VeloCloneKiller, VeloClusterTrackingSIMD, VeloClusterTrackingSIMDFull, VeloKalman, @@ -460,9 +461,13 @@ def make_VeloClusterTrackingSIMD_tracks(make_tracks=make_VeloClusterTrackingSIMD """ tracking = make_tracks() + fwd_noclone = VeloCloneKiller( + InputTracksLocation=tracking.TracksLocation, HitsLocation=tracking.HitsLocation + ) return { - "Pr": tracking.TracksLocation, + "Pr": fwd_noclone.OutputTracksLocation, # tracking.TracksLocation, "Pr::backward": tracking.TracksBackwardLocation, + "ForwardCloneKilled": fwd_noclone.OutputTracksLocation, } diff --git a/Hlt/RecoConf/python/RecoConf/standalone.py b/Hlt/RecoConf/python/RecoConf/standalone.py index 79c38698828..586bc255ecb 100644 --- a/Hlt/RecoConf/python/RecoConf/standalone.py +++ b/Hlt/RecoConf/python/RecoConf/standalone.py @@ -705,7 +705,7 @@ def standalone_hlt2_reco( # "VP" also has the other banks. The proper fix would be to # have the RawBankSizeMonitor accept multiple RawEvents (or # even RawBank::View's) - RawBankSizeMonitor(RawEventLocation=default_raw_event("VP")) + # RawBankSizeMonitor(RawEventLocation=default_raw_event("VP")) ] data += [monitor_muon_PID(muon_pids)] -- GitLab From cf58d06230892c60edde6b0f45105e8086584208 Mon Sep 17 00:00:00 2001 From: Arthur Hennequin Date: Tue, 8 Jul 2025 12:49:16 +0200 Subject: [PATCH 2/2] Cleanup --- Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py | 2 +- Hlt/RecoConf/python/RecoConf/standalone.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py b/Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py index 5561ca3d421..e49d502e31b 100644 --- a/Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py +++ b/Hlt/RecoConf/python/RecoConf/legacy_rec_hlt1_tracking.py @@ -465,7 +465,7 @@ def make_VeloClusterTrackingSIMD_tracks(make_tracks=make_VeloClusterTrackingSIMD InputTracksLocation=tracking.TracksLocation, HitsLocation=tracking.HitsLocation ) return { - "Pr": fwd_noclone.OutputTracksLocation, # tracking.TracksLocation, + "Pr": fwd_noclone.OutputTracksLocation, "Pr::backward": tracking.TracksBackwardLocation, "ForwardCloneKilled": fwd_noclone.OutputTracksLocation, } diff --git a/Hlt/RecoConf/python/RecoConf/standalone.py b/Hlt/RecoConf/python/RecoConf/standalone.py index 586bc255ecb..79c38698828 100644 --- a/Hlt/RecoConf/python/RecoConf/standalone.py +++ b/Hlt/RecoConf/python/RecoConf/standalone.py @@ -705,7 +705,7 @@ def standalone_hlt2_reco( # "VP" also has the other banks. The proper fix would be to # have the RawBankSizeMonitor accept multiple RawEvents (or # even RawBank::View's) - # RawBankSizeMonitor(RawEventLocation=default_raw_event("VP")) + RawBankSizeMonitor(RawEventLocation=default_raw_event("VP")) ] data += [monitor_muon_PID(muon_pids)] -- GitLab