BandQ recover events for Xib

This is a bug fix related to the discussion at: lhcb-datapkg/AnalysisProductions!2030 (comment 8705113)

The B-for-spectroscopy hlt2 full-stream line and sprucing line share the same builder. The Xib mass window in the builder is wrong in 2024 data taking. In principle the bug will be there and cannot be fixed for 2024 hlt2 output.

When developing the EOY bug fixes in !3839 (merged) , people noticed the issue and update the builder accordingly. However, people were not aware of the existence of the hlt2 filtering in the corresponding sprucing line. The hlt2 filtering will still contain the wrong Xib mass window cut (>5780MeV), and kill about 50% of Xib signals .

This MR is mainly for removing the hlt2 filter of the Xib spectroscopy sprucing lines, and rely on other physics full stream lines to recover the efficiency.

Besides, the make_xib2cx_for_spectroscopy default AM cut is not updated in the BandQ central MR. Try to update it together here.

assigned to @mengzhen and @yajing

added 1 commit

• 6ee5a855 - Fixed formatting

Compare with previous version

mentioned in merge request lhcb-datapkg/AnalysisProductions!2030 (merged)

added RTA label

@mengzhen @yajing Is this supposed to be a patch for EoY Sprucing?

added Spruce_EoY24 label

added BandQ label

added PR/Moore_spruce_bandwidth label

added PR/full_throughput_test label

I did another test, with the hlt2 filter of Xibm_for_spectroscopy_sprucing_line and Xib0_for_spectroscopy_sprucing_line directly removed. The output size of the dst file almost does not change (keeps 900MB). Then my prediction is wrong.

The rates of the Xib lines do change a bit. Xib0: 326 events fired -> 332; Xibm: 172 -> 188.

Will tupling the output dst to understand whether the low-mass region events are recovered.

added 1 commit

• a3b8b06d - Remove all filters

Compare with previous version

added 1 commit

• 3de55976 - Fix bug

Compare with previous version

changed the description

/ci-test

added ci-test-triggered label

• [2024-11-08 14:57] Validation started with lhcb-2024-patches-mr#1834
• [2024-11-08 17:30] Validation started with lhcb-2024-patches-mr#1837

Update of the local test: for Xib2JpsiX decays the event rate with mass < 5.78GeV is increased, but for hadronic modes nothing changes.

I think the reason is: https://gitlab.cern.ch/lhcb/Moore/-/blob/2024-patches/Hlt/Hlt2Conf/python/Hlt2Conf/lines/bandq/builders/b_for_spectroscopy.py#L573 The value of 5780 is still there, and used later in https://gitlab.cern.ch/lhcb/Moore/-/blob/2024-patches/Hlt/Hlt2Conf/python/Hlt2Conf/lines/bandq/builders/b_for_spectroscopy.py#L787 et. al. as the default setting.

Will do another round of local test with the issue fixed and add a new commit if everything looks fine.

changed the description

added 1 commit

• 48e0bcae - fix default xib mass window

Compare with previous version

marked this merge request as ready