Velo-SciFi track reconstruction based on SciFi seeding and Matching algorithms
Goes together with Moore!1733 (merged)
Long track reconstruction based on SciFi seeding and Velo-SciFi matching. Information from the UT is not included for the moment. The algorithms can be found in:
-
device/SciFi/hybridseeding
for the SciFi seeding. The SciFi standalone reconstruction is based on two main algorithms, theseed_xz
andseed_confirmTracks
. The HLT2 implementation of the HybridSeeding can be found here: https://gitlab.cern.ch/lhcb/Rec/-/blob/master/Pr/PrAlgorithms/src/PrHybridSeeding.cpp -
device/track_matching
for the Velo-SciFi matching. The main reconstruction algorithm istrack_matching_veloSciFi
and is adapted from HLT2 (https://gitlab.cern.ch/lhcb/Rec/-/blob/master/Pr/PrAlgorithms/src/PrMatchNN.cpp )
The output of the track-matching algorithm is long tracks and has been adapted to have the same format as for the forward algorithm, in order to facilitate integration in the HLT1 sequence. The hlt1 reconstruction with seeding+track-matching can be enabled by using the hlt1_pp_matching
or hlt1_pp_matching_validation
sequences.
More information on the current strategy and performance of the SciFi seeding + matching algorithms can be found in the 17/01/2022 WP2 meeting: https://indico.cern.ch/event/1116110/
Alternative (original) parallelisation scheme for the SciFi seeding can be found in https://gitlab.cern.ch/lhcb/Allen/-/tree/lohenry-uv-2, with the first complete porting to GPU by @bjashal and @jzhuo in https://gitlab.cern.ch/lhcb/Allen/-/tree/HLT1Seeding_v1 . Both branches are kept to allow development on that parallelisation scheme.
Developments on the UT-seeding and UT-SciFi matching are ongoing in: https://gitlab.cern.ch/lhcb/Allen/-/tree/dev_trackmatching_christina
Merge request reports
Activity
added RTA label
- Resolved by Christina Agapopoulou
From the first round of nightly tests this is the like-for-like comparison on the v6 SciFi dataset. (Actually given the low-momentum efficiencies the real like-for-like comparison would be with the forward without cuts, but let's leave that aside for now) Looks very positive. The reduction in performance on CPU is maybe something to think about but not to me a huge deal.
@dcampora @dovombru @decianm given the unexpectedly positive results I'd appreciate an expedited review here with a focus on figuring out whether there is any major mistake/bug which spoil the story and would prevent this being used. We need to be in a position to report on this in LHCb week.
Seeding + Track matching
NVIDIA GeForce RTX 3090 │████████████████████████████████████████████████ 244.16 kHz NVIDIA RTX A6000 │███████████████████████████████████████████████ 239.77 kHz NVIDIA RTX A5000 │██████████████████████████████████████ 190.43 kHz NVIDIA GeForce RTX 2080 Ti │████████████████████████████████ 160.37 kHz AMD EPYC 7502 32-Core │██ 14.79 kHz Intel Xeon E5-2630 v4 │▌ 3.25 kHz ┼────┴────┼────┴────┼────┴────┼────┴────┼────┴────┼ 0 50 100 150 200 250
Velo + VeloUT + Forward
NVIDIA GeForce RTX 3090 │████████████████████████████████████████████ 223.53 kHz (1.00x) NVIDIA RTX A6000 │███████████████████████████████████████████ 215.68 kHz (1.00x) NVIDIA RTX A5000 │███████████████████████████████████ 178.03 kHz (1.00x) NVIDIA GeForce RTX 2080 Ti │████████████████████████████ 143.02 kHz (1.00x) MI100 │██████████████████ 94.57 kHz (1.02x) AMD EPYC 7502 32-Core │███ 16.34 kHz (1.00x) ┼────┴────┼────┴────┼────┴────┼────┴────┼────┴────┼ 0 50 100 150 200 250
- Resolved by Christina Agapopoulou
@maxime would it be possible to test whether this has any negative impact on the track-cluster matching, electron reconstruction, and brems recovery enabled in !565 (merged) ?
- Resolved by Christina Agapopoulou
The automatic output rate tests on minimum bias samples confirm the increase in TrackMVA rate shown in the WG presentation. As @sstahl pointed out this is unexpected and could indicate a bug in one of the quantities this line cuts on -- PT, track chi2, or IPCHI2. Hints are that no big increase is seen for the two track line and TrackMVA does not seem to increase the output rate on a Bs2PhiPhi sample.
While investigating this I would suggest to run some line independence tests turning off everything except the TrackMVA line (or the TwoTrackMVA line, or ...), because we anyway need to run those before datataking to make sure we haven't introduced any framework bugs over the past months so we might as well try to save some time by dealing with that as a side-effect of figuring this out.
mentioned in issue Moore#375 (closed)
added 1 commit
- e19955f7 - fixed velo indexing bug for ParKalmanVeloOnly in hlt1_pp_default sequence
added 1 commit
- 7da96bfb - added electron_fromB categories in matching checker
mentioned in merge request !744 (closed)
mentioned in merge request !741 (merged)
mentioned in merge request !764 (merged)
mentioned in issue Moore#382 (closed)
added 1 commit
- 165edf0f - kalman matching validator and split matching sequence to v4/v6
mentioned in issue Moore#390 (closed)
added 302 commits
-
cedf7bd9...cdab1533 - 299 commits from branch
master
- a1d20fa8 - Merge with master, does not compile yet
- 5ea1bbac - Solving compilation issues
- 18a2ed49 - Finished the merge
Toggle commit list-
cedf7bd9...cdab1533 - 299 commits from branch
mentioned in merge request !690 (merged)
mentioned in merge request !734 (merged)
added 1 commit
- cd1ce8e3 - Remove raw_bank_version as argument of decode_scifi
added 129 commits
-
cd1ce8e3...0b12a15d - 127 commits from branch
master
- 7699bdff - Merge with master
- c62aae18 - Fixed compilation
-
cd1ce8e3...0b12a15d - 127 commits from branch
mentioned in issue Moore#411 (closed)
added 273 commits
-
8e71c161...526a48d9 - 271 commits from branch
master
- 32d44ff8 - Merge remote-tracking branch 'origin/master' into matching_veloscifi_hlt1
- faf3ed2b - Adapt matching to new scifi views, cleanup
-
8e71c161...526a48d9 - 271 commits from branch
added 1 commit
- 97c12065 - Use scifi views to access velo indices in MomentumBremCorrection, remove ut from matching sequence
added 1 commit
- 7c76b9ab - Use scifi views to access velo indices in MomentumBremCorrection, remove ut from matching sequence
added 1 commit
- 68d535a9 - Remove unused files, fix warnings, fix matching_nolines sequence config
added 1 commit
- 143070c7 - Remove unused files, fix warnings, fix matching_nolines sequence config
- A deleted user
added hlt1-throughput-decreased label
- A deleted user
removed hlt1-throughput-decreased label
added 1 commit
- b166bb13 - Cleanup of UT event model and unused sequences
added 1 commit
- d4e7d60c - Cleanup of UT event model and unused sequences
- A deleted user
added hlt1-throughput-decreased label
mentioned in issue Moore#416 (closed)