Add B2ll sprucing lines and fix B2llgamma

This MR adds sprucing lines for B2ee (+SameSign), B2mumu (+SameSign), B2emu (+SameSign) and B2hh. Following comments I got the extra outputs are removed and only persist reco is used to avoid duplication (no info loss expected). I also fixed a bug in the B2llgamma lines (I used the B IPCHI2) and applied hard cuts to keep the rates manageable. Will have to be retuned in the future probably after photon ID has improved, but for now should be fine.

The rates below have been recalculated on the full upgrade_minbias_hlt1_filtered sample after the muon ID bug has been fixed.

----------------------------------------------------------------------------------------------------
INFO:    Starting /scratch03/titus.mombacher/HLT2Dev/stack/MooreAnalysis/HltEfficiencyChecker/scripts/hlt_calculate_rates.py...
----------------------------------------------------------------------------------------------------
INFO:    No lines specified. Defaulting to all...
HLT rates: 
----------------------------------------------------------------------------------------------------
Line:    Hlt2RD_BToEEGamma_LineDecision          Incl: 0.13348461589801777 +/- 0.09 kHz, Excl: 0.13348461589801777 +/- 0.09 kHz
Line:    Hlt2RD_BToEE_LineDecision               Incl: 0.06674230794900889 +/- 0.06 kHz, Excl: 0.0  +/- 0.0  kHz
Line:    Hlt2RD_BToEE_SameSign_LineDecision      Incl: 0.06674230794900889 +/- 0.06 kHz, Excl: 0.0  +/- 0.0  kHz
Line:    Hlt2RD_BToMuEGamma_LineDecision         Incl: 0.06674230794900889 +/- 0.06 kHz, Excl: 0.06674230794900889 +/- 0.06 kHz
Line:    Hlt2RD_BToMuE_LineDecision              Incl: 0.0  +/- 0.0  kHz, Excl: 0.0  +/- 0.0  kHz
Line:    Hlt2RD_BToMuE_SameSign_LineDecision     Incl: 0.0  +/- 0.0  kHz, Excl: 0.0  +/- 0.0  kHz
Line:    Hlt2RD_BToMuMuGamma_LineDecision        Incl: 0.0  +/- 0.0  kHz, Excl: 0.0  +/- 0.0  kHz
Line:    Hlt2RD_BToMuMu_LineDecision             Incl: 0.0  +/- 0.0  kHz, Excl: 0.0  +/- 0.0  kHz
Line:    Hlt2RD_BToMuMu_SameSign_LineDecision    Incl: 0.0  +/- 0.0  kHz, Excl: 0.0  +/- 0.0  kHz
Line:    Hlt2RD_BTohh_LineDecision               Incl: 0.26696923179603554 +/- 0.13 kHz, Excl: 0.20022692384702662 +/- 0.11 kHz
----------------------------------------------------------------------------------------------------
Hlt2 Total:                                      Rate:   0 +/- 0.2 kHz
----------------------------------------------------------------------------------------------------```

added RD hlt2 labels

marked this merge request as draft

assigned to @tmombach

requested review from @msaur, @fevolle, @htilquin, and @hvc

added RTA selection labels

Hi @tmombach, before any review, can you fix pipelines? Some unused imports and similar. Thanks.

(New Sprucing lines. Work related to DPA task https://gitlab.cern.ch/lhcb-dpa/project/-/issues/78.)

Hello Titus, thank you for this update. I have a minor comment concerning the file names. Would you mind to rename your hlt 2 file name into something like hlt2_B2ll.py and the sprucing one into spruce_... then it would be similar to the other Hlt2 + sprucing files.

added 1 commit

• 0afe119e - remove and fix unused and broken imports

Compare with previous version

added 1 commit

• 43b05d2b - rename files to better fit to scheme

Compare with previous version

changed the description

mentioned in issue Rec#208 (closed)

added 1 commit

• fad26cd7 - Fixed formatting

Compare with previous version

Hi @tmombach , I am not sure I understand why your lines go to sprucing. By persist reco to allow isolation developments, do you mean that you plan to use isolation variables in your lines, or that you plan to work on some isolation code in the future?

added 1 commit

• 43864df2 - move to standard particle vertex fitter, which now seems to work

Compare with previous version

changed the description

now don't expect any more changes (unless requested by reviewers)

approved this merge request