SL: SLB lines as sprucing with persitreco=False

This MR follows the discussion. Here we duplicate all SLB Hlt2 lines with persistreco=True (essentially all of them) into sprucing lines with persitreco=False.

To do:

• After discussion with WG turn the persistreco=True as true for select few lines for which the isolation studies need to be conducted.
• Add a OR for Hlt2Topo2BodyDecision || Hlt2Topo3BodyDecision || <corresponding_line_in_hlt2>. This would be helpful for overlap studies.

FYI: @jugarcia , @sklaver , @mvesteri , @poluekt, @gciezare

Keeping track of Sprucing selections in DPA task https://gitlab.cern.ch/lhcb-dpa/project/-/issues/78. https://gitlab.cern.ch/lhcb-dpa/project/-/issues/243

added SLB Line label

requested review from @nskidmor and @anmorris

assigned to @amathad

@nskidmor : These are essentially duplicates for the Hlt2 lines. Let me know if there is some optimisation we can do here e.g. should we add filter on the respective hlt2 lines (with hlt2_filter_code)?

changed the description

changed the description

added RTA label

added 1 commit

• 474b823f - Fixed formatting

Compare with previous version

mentioned in merge request !2342 (merged)

added 1 commit

• 8bb9a39c - add hlt2_filter_code

Compare with previous version

resolved all threads

added 1 commit

• 134b0b9f - Fixed formatting

Compare with previous version

@amathad, @nskidmor, great to see the filters to be efficient . This reminds me - would it make sense to add a warning to the SpruceLine class in case no filters are given? I mean, in principle one should always run a Sprucing line on some HLT2 line (certainly for the bulk of the use-cases) at least, hence good to hint.

resolved all threads

added hlt2 label

resolved all threads

resolved all threads

added 1 commit

• 5d3119d6 - Eduardo's suggestion

Compare with previous version

added 52 commits

• 5d3119d6...32f59261 - 51 commits from branch master
• acadc73f - Merge branch 'master' of ssh://gitlab.cern.ch:7999/lhcb/Moore into AM_spruce

Compare with previous version

added 1 commit

• 7167673b - add rjsi spruce lines

Compare with previous version

mentioned in merge request !2068 (merged)

added selection label

added June-2023-data label

removed June-2023-data label

changed the description

added Spruce_June23 label

mentioned in merge request !2347 (merged)

added 3 commits

• 7167673b...1a791258 - 2 commits from branch master
• a17e3184 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/lhcb/Moore into AM_spruce

Compare with previous version

added 56 commits

• a17e3184...66fe3dd2 - 55 commits from branch master
• af7eae2d - Merge branch 'master' of ssh://gitlab.cern.ch:7999/lhcb/Moore into AM_spruce

Compare with previous version

added 30 commits

• af7eae2d...131f3ad0 - 20 earlier commits
• a152dd20 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/lhcb/Moore into SLB_AM_KSLambda
• f9736edc - add persistreco
• 79265905 - apply Suzanne's comment
• 6e5a0e20 - Rename lines based on https://codimd.web.cern.ch/1FARROp8R1mdd5zCHLktlA?view
• 38842fc3 - undo changes to lines which will be removed by l anyway
• 439ce1a5 - undo changes to lines which will be removed by MR2347 anyway
• 3a65b069 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/lhcb/Moore into SLB_AM_KSLambda
• 0d997190 - name changes
• 1c03155b - Merge branch 'anmorris_SLB_naming_convention' of...
• 74ddd775 - Merge branch 'SLB_AM_KSLambda' of ssh://gitlab.cern.ch:7999/lhcb/Moore into AM_spruce

Compare with previous version

marked the checklist item After discussion with WG turn the persistreco=True as true for select few lines for which the isolation studies need to be conducted. as completed

added 5 commits

• a90034de - Unified the line builders for the B+ -> D0 tau nu modes.
• 751dd5cb - Fixed a decay descriptor.
• 10b17558 - Removed an obsolete input argument for a composite builder.
• c51f35ec - Merge branch 'SLB_remove_duplicated_lines_new' of...
• 72ba4ad6 - persistreco false

Compare with previous version

changed the description

added 2 commits

• d98fd851 - Remove Vub that had been missed
• 24802d1a - Merge branch 'anmorris_SLB_naming_convention' of...

Compare with previous version

added 1 commit

• 1be2d2a6 - add persistreco is True

Compare with previous version

added 1 commit

• 58955ccb - Fixed formatting

Compare with previous version

requested review from @abertoli

Hi @abertoli and @nskidmor , This MR sits on top of the three Moore MRs that have been tested and due to be merged since weeks. I hope they all go in together.

PS: I will be on paternity leave for two weeks starting tomorrow. So please contact Andy or Julian or Suzane in case of emergencies.

assigned to @yagao

@yagao : Could you please trigger a test with the above dependencies?

marked this merge request as ready

/ci-test !2430 (merged) !2068 (merged) !2347 (merged)

added ci-test-triggered label

• [2023-06-19 15:29] Validation started with lhcb-master-mr#8286
• [2023-06-20 10:45] Validation started with lhcb-master-mr#8300

added hlt2-throughput-decreased label