Add monitoring histos to BdToDsmK_DsmToHHH_spucing_FEST_line

Adding Monitoring histograms of the mass of the motherparticle to the following sprucing lines

• BdToDsmK_DsmToHHH_spucing_FEST_line

This line was chosen, instead of eg B2Dpi as considered previously, because it is booked individually and not via for loop which would have meant implementing histograms for all default spruce lines.

For developing of SprucingDQ. @shunan @abertoli

Work on DPA task lhcb-dpa/project#211

added DPA-WP1 label

requested review from @nskidmor

assigned to @eleckste

added RTA label

Hello @eleckste, for this and future work can you kindly link MRs (and issues) to the relevant DPA task(s) so that we keep record of work and know where to look for "anything"? Many thanks.

[This MR is so far empty but I guess you know already and will be pushing code soon.]

added 1 commit

• 5d2ec1e1 - add B mass monitor to BdToDsmK_DsmToHHH_sprucing_FEST_line

Compare with previous version

changed the description

added 1 commit

• c099b4f3 - fix formatting

Compare with previous version

changed title from Draft: Add monitoring histos to Bd2DPi, Lb2LcPi and Bs2DsPi spruce lines to Draft: Add monitoring histos to BdToDsmK_DsmToHHH_spucing_FEST_line

changed the description

@eleckste have you tested this locally and the hist looks good? :)

@eleckste looking here: https://gitlab.cern.ch/lhcb/Moore/-/compare/master...eleckste-sprucing-monitoring I see that just a few changes to the code are required

if you ran Moore on some min. bias upgrade MC the mass shape will look flat or decreasing

from the differences in the code above I do not get where the histogram is available

does Moore dump a .root file with the histograms requested ?

approved this merge request

added 1 commit

• acc9809c - added histo_file option to yamls

Compare with previous version

added 1 commit

• 48aa43e2 - fix formatting

Compare with previous version

added 1 commit

• 0a5332d8 - Fixed formatting

Compare with previous version

marked this merge request as ready

Hi @frodrigu this is ready for testing :)

/ci-test

@eleckste Please look at the pipeline test results. The build-docs check is failing and needs to be addressed.

assigned to @frodrigu

added ci-test-triggered label

• [2023-03-24 14:52] Validation started with lhcb-master-mr#7428
• [2023-03-25 00:07] Validation started with lhcb-gaudi-head#3513

resolved all threads

added lhcb-gaudi-head label

resolved all threads

removed lhcb-gaudi-head label

unassigned @frodrigu

@sponce ci-test and nightly tests are ok. Pipeline has a fail in the doc, but Nicole commented !2190 (comment 6564928).

resolved all threads

merged

mentioned in commit 5f746709