New HLT2 sequences for PbPb data taking with and without UT

• creating a new specific sequence for ion-ion data taking (PbPb and Pb gas)

  • adding two new sequences to standalone.py to be used during PbPb data-taking: standalone_hlt2_light_reco_ion (created from standalone_hlt2_light_reco) and standalone_hlt2_light_reco_ion_without_UT (created following standalone_hlt2_light_reco_without_UT)

  • each of them call (in hlt2_tracking.py) make_hlt2_tracks_ion and make_hlt2_tracks_ion_without_UT (instead of make_hlt2_tracks and make_hlt2_tracks_UT) to get the track information.

    • In them, Best tracks can be filtered with LoKi cut throught a LoKiTrackSelector. The cut can be implemented through the bind fit_preselection, to be set to None for no cut and directly to the selection string. Only best tracks are filtered (BestLong in reconstruction without UT and both BestLong and BestDownstream in reconstruction with UT). The cut must be tuned to a loose ghost probability cut (like TrGHOSTPROB<0.9999) to reduce the amount of fake tracks in high occupancy events to greatly reduce the time spent in RICH reconstruction.

    • adding 8 new option files in Moore/Hlt/RecoConf/options for testing, with and without MC checking: hlt2_lead_lead_light_reco_pr_kf_without_UT_gec_40000.py, hlt2_lead_lead_light_reco_pr_kf_without_UT_gec_60000.py, hlt2_lead_lead_light_reco_pr_kf_gec_40000.py, hlt2_lead_lead_light_reco_pr_kf_gec_60000.py.

    • In the test option file for the sequence with UT, we set a cut of TrGHOSTPROB<0.9999 to reduce the time spent at RICH (following this presentation). For the test of the sequence without UT, we use TrGHOSTPROB<0.8. This is tighter with respect to what was shown in the presentation since the ghost probability performance is now worse in the reconstruction without UT (see M!1560 for details).

  • future tunings of the track reconstruction algorithms (specially seeding and forward) to improve the performance in high-occupancy events will be called only in those sequences to avoid interference with pp conditions.

added RTA RTA WP2 hlt2 labels

assigned to @baudurie and @oboenteg

changed the description

added 122 commits

• aa87b835...318fbcd7 - 121 commits from branch master
• bc06664b - Merge branch 'master' into RTA_audurier

Compare with previous version

added 1 commit

• ba28b1bf - Add new option files to test new sequence

Compare with previous version

added 1 commit

• 5a5a7bd9 - Fixed formatting

Compare with previous version

changed title from Draft: New sequences for PbPb data taking with and without UT to Draft: New HLT2 sequences for PbPb data taking with and without UT

changed the description

@oboenteg, I succeed in running your files. I guess we can proceed with the request.

marked this merge request as ready

gentle ping, @cmarinbe, in case you missed this amazing merge request

mentioned in issue #448 (closed)

mentioned in issue #449 (closed)

added 1 commit

• 8de8136e - Simplified make_hlt2_tracks_ion and make_hlt2_tracks_ion_without_UT, added option files GEC 50000

Compare with previous version

added 1 commit

• 2580e2e1 - Remove unused TrackListRefiner from hlt2_tracking.py imports

Compare with previous version

added 1 commit

• 01ffe4fc - Fixed formatting

Compare with previous version

added 1 commit

• dc40505e - Changed fit_preselection to post_fit_selection

Compare with previous version

resolved all threads

/ci-test

added ci-test-triggered label

• [2022-07-04 17:03] Validation started with lhcb-master-mr#4936
• [2022-07-05 15:08] Validation started with lhcb-master-mr#4945
• [2022-07-06 10:34] Validation started with lhcb-master-mr#4954

added 1 commit

• 70d3ed27 - Removed mult_checkers from standalone_hlt2_light_reco

Compare with previous version

/ci-test

resolved all threads

approved this merge request

@oboenteg can you please rebase/cleanup history of this MR. I cannot squash it anymore and the history is full of useless commits (fixed formatting, merged master into this branch, etc...)

By the way, merging master into your branch is one of the reasons why we cannot squash anymore (and why you will probably suffer when rebasing). You may want to use rebase instead. In case you are not at ease with all this, see the git tutorial in https://twiki.cern.ch/twiki/bin/view/LHCb/LHCbSoftwareTutorials

added 1 commit

• 31ac4012 - fix option files

Compare with previous version

added 1 commit

• 061c5812 - write specific PbPb sequence

Compare with previous version

added 217 commits

• 061c5812...318fbcd7 - 215 commits from branch master
• c0706faf - keep PrMultiplicityChecker only in HLT2 ion sequences
• b82577ab - write specific PbPb sequence

Compare with previous version

/ci-test

resolved all threads

• [2022-07-11 12:04] Validation started with lhcb-master-mr#4987
• [2022-07-12 17:59] Validation started with lhcb-master-mr#5011

added 125 commits

• b82577ab...714190c8 - 123 commits from branch master
• d8bbbfe9 - keep PrMultiplicityChecker only in HLT2 ion sequences
• 2d0a0944 - write specific PbPb sequence

Compare with previous version

Hi @sponce, I had to rebase on master the branch today again, I noticed that some commits were missed (I guess because yesterday I forgot to update my local version of master before rebasing..). Sorry, let's hope this is the good one!

resolved all threads

/ci-test

resolved all threads

merged

mentioned in commit dd862cd7

mentioned in issue #454 (closed)

mentioned in merge request !1795 (closed)