KstG/PhiG high-PT lines for photon pid

Calibration lines for neutral PID following LHCb-INT-2021-002 except several things (track ghost prob, ProbNN, ...) commented for early data-taking. Plus we can't vertex combinations including neutrals atm, so all vertexing pushed to the charged combinations.

Following four modes are included atm and tested with 14983 minbias events with the retentions:

• Bd->KstG 10/14983
• Bs->PhiG 1/14983
• Eta->mumuG 0/14983
• Dst->D0(->Kpipi0)pi 0/14983

For the moment only the KstG/PhiG are being included. The other modes will be included in a separate MR.

requested review from @smalde, @cmarinbe, @chefdevi, @isanders, and @alobosal

changed title from Draft: committed three lines for high/low pt photon + pi0 to Draft: committed three lines for high/low pt photon + pi0 PID calibration lines

changed the description

added RTA label

Hello @bdey. Very nice to see this coded. Fast comments:

• for D0->Kpipi0, why using a mass window on the Kpi?
• for MuMuG, given you have 0 events, will you consider lowering the gamma PT cut (currently set to 1 GeV)?

Apologies for the D0, just found this: https://gitlab.cern.ch/lhcb/Moore/-/blob/bdey_pid_neutral2/Hlt/Hlt2Conf/python/Hlt2Conf/lines/pid/utils/neutral_pid.py#L115

@bdey Is there a conversion of the retention to how that translates to Hz for the lines? I think you will have to remove the draft tag for the merge to be looked at more closely.

hi, @smalde, for Hlt1 filtered the retention also gives the rate in MHz, so for KstG, it's 0.66kHz and for PhiG, it's 0.06kHz.

The Kst mass window is +/- 200 MeV and Phi mass window is +/- 10 MeV. I guess the Phi can be loosened a little bit.

For the other two, I'm running on more events. Plus we also need new signal samples for these.

New (xdigi) samples to be requested:

• eta_mumugamma=TightCut.dec (39112230): the cuts seem exactly aligned to what's in the line. Create new decfile with DecProdCut?

• Dst_D0pi,Kpipi0=cocktail,DecProdCut.dec (27263400)

• Ds+_etaprimepi,rhogamma=DecProdCut.dec (23103200): some xdigi samples exist, but we can ask more. Decfile seems ok.

• Dsst_Dsgamma,KKpi=DecProdCut.dec (27163221): some samples exist.

added 1 commit

• 1f376ea5 - fixed Phi mass range to [1000,1040]

Compare with previous version

added 16 commits

• dba031a7 - Update D0->KsKs HLT2 lines using the new flag to select LD candidates
• 45995cce - Rename TrackPV2HalfAlignMonitor to TrackPV2HalfAlignMonitor
• a1f62abb - Update References for: Rec!2904 (merged), !1626 (merged) based on lhcb-master-mr/4840
• 69dccb3d - Added line about setting allen_hlt1 to True to run on HLT1 output
• 85949804 - fixed formatting of code snippets
• 85a7bd13 - fixed python linting
• 28ffc44d - added back in mistakenly deleted sequences import
• 19d82476 - Fixed formatting
• 94fa5aa1 - removed added files not relevant to this MR
• 18672f8b - Fix option files that include GetFlavourTaggingParticles
• 1fb20fc8 - Test running Allen sequence on XDST
• c105ebb6 - Easier configuration of Allen sequence by name.
• 43bca132 - Dump Rich hits in phoenix json files
• 2ca4b12a - Update References for: LHCb!3659 (merged), !1634 (merged) based on lhcb-master-mr/4917
• 538acb66 - Temporary fix for references of the hlt2_protoparticles_fastest test
• ab548cd5 - Update References for: Allen!691 (merged) based on lhcb-master-mr/4905

Compare with previous version

added 1 commit

• 8b754d4a - Kst mass window 150 MeV and loosed Kpi swap cut

Compare with previous version

@bdey can you remove the draft tag so that this can be reviewed by the shifter

marked this merge request as ready

hi @smalde the KstG/PhiG lines will be slightly modified. However, the eta2mumug and pi0 lines don't have anything passing thru' even after running on larger minbias samples, so they need further checks. I will probably remove those for the moment so that we can merge the KstG/PhiG.

@bdey - ok sorry i thought this was ready for merging. For the modifications on KstG/PhiG are these modifications prior to data-taking, or post? If prior i'd like for these changes to have high priority and done before the line is merged. If post then yes i agree remove the potential problematic lines so that the KstG/PhiG line can be merged.

assigned to @bdey

@bdey Would be good to rebase, solve merge conflicts and clean up what all should be merged / reviewed. I don't think you want to merge all the things you have in this branch.

added 1 commit

• de832870 - removed EtaMuMuG/DstD0KPiPi0 and modified KstG/PhiG

Compare with previous version