Charm HLT2 lines related to detection asymmetries

This MR adds HLT2 lines related to Cabibbo-favoured modes to measure 2-track asymmetries. They are close to the equivalent of what we had in Run 1/2. The main principle is to keep the selection separate from the standard lines, as we typically require lower pT cuts, but so far used a cut on the IPCHI2 in the trigger to lower the rate.

All of the cuts are basically untuned, but all of the particle combinations are kept separate to allow for tuning later.

Edit mstahl: Rate increase with other charm lines 1.6 kHz. Inclusive rates:

Hlt2Charm_LcpToPpKmPip_ADet_Line    #=14983   Sum=16   Eff=|(0.1067877 +- 0.0266827)%|
Hlt2Charm_LcpToKsPp_LL_ADet_Line    #=14983   Sum=0    Eff=|( 0.000000 +- 0.00000 )%|
Hlt2Charm_DspToPimPipPip_ADet_Line  #=14983   Sum=13   Eff=|(0.08676500 +- 0.0240538)%|
Hlt2Charm_DspToKsKp_LL_ADet_Line    #=14983   Sum=0    Eff=|( 0.000000 +- 0.00000 )%|
Hlt2Charm_DspToKmKpPip_ADet_Line    #=14983   Sum=14   Eff=|(0.09343923 +- 0.0249610)%|
Hlt2Charm_DpToKsPip_LL_ADet_Line    #=14983   Sum=0    Eff=|( 0.000000 +- 0.00000 )%|
Hlt2Charm_DpToKmPipPip_ADet_Line    #=14983   Sum=8    Eff=|(0.05339385 +- 0.0188725)%|

added Charm selection labels

added 1 commit

• 721bbbbe - Added lines.

Compare with previous version

added 1 commit

• fd4c954c - Updated formatting

Compare with previous version

added RTA label

added 1 commit

• b706f8d6 - Linter comments

Compare with previous version

added 1 commit

• e46e2d58 - Added D+ lines comment

Compare with previous version

added 1 commit

• 17e4c69a - Formatting again

Compare with previous version

added hlt2 label

added 1 commit

• e3c45df7 - Now it actually works

Compare with previous version

Rates in the current state (on upgrade_minbias_hlt1_filtered):

  LAZY_AND: Hlt2CharmDpToKmPipPip_ForKpiAsym_Line                                                             #=14983   Sum=164         Eff=|( 1.094574 +- 0.0850028)%|

   LAZY_AND: Hlt2CharmDpToKsLLPip_ForKpiAsym_Line                                                              #=14983   Sum=1           Eff=|(0.006674231 +- 0.00667401)%|

   LAZY_AND: Hlt2CharmDspToKmKpPip_ForKpiAsym_Line                                                             #=14983   Sum=78          Eff=|(0.5205900 +- 0.0587916)%|

   LAZY_AND: Hlt2CharmDspToKsLLKp_ForKpiAsym_Line                                                              #=14983   Sum=5           Eff=|(0.03337115 +- 0.0149215)%|

   LAZY_AND: Hlt2CharmDspToPimPipPip_ForpipiAsym_Line                                                          #=14983   Sum=42          Eff=|(0.2803177 +- 0.0431933)%|

   LAZY_AND: Hlt2CharmLcToKsLLproton_ForKpiAsym_Line                                                           #=14983   Sum=2           Eff=|(0.01334846 +- 0.00943816)%|

   LAZY_AND: Hlt2CharmLcToprotonKmPip_ForKpiAsym_Line                                                          #=14983   Sum=61          Eff=|(0.4071281 +- 0.0520212)%|

As noted in the code, the rate of this line is controllable because of a cut on the IP of the D, which is currently not yet possible.

@ldufour please assign me when this is ready for testing

mentioned in issue #382 (closed)

added 1 commit

• 87c45da3 - Disable track chi2 cut

Compare with previous version

added 1 commit

• 91d59fee - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 99 commits

• 91d59fee...8dc02754 - 81 commits from branch master
• f9261839 - move D0 -> KS0 KS0 lines to ThOr
• b9296765 - Charm: naming changes
• 55c88404 - ThOr migration of D -> hhh
• 819c660c - ThOr migration of prompt D0->hh lines
• dbcf2270 - HLT2 lines to monitor HLT1 KS lines
• 5590f3be - Add lines for Early Measurement Charm Cross-Sections
• 304237f0 - [charm/builders] fix errors introduced in !1304 (merged)
• 9dafd986 - Charm HLT2 lines: DpToKSh
• 75d20284 - Charm HLT2 lines: LcTophh
• dc32f720 - Added lines.
• cd58f994 - Updated formatting
• a04b6052 - Linter comments
• d2778a0e - Added D+ lines comment
• 0ff8005d - Formatting again
• 81c3ec61 - Now it actually works
• 10b99e5c - Disable track chi2 cut
• 72f1860a - Apply 1 suggestion(s) to 1 file(s)
• 2b000c1c - [charm/detection_asymmetry_lines] update line and combiner names, add functors...

Compare with previous version

changed the description

requested review from @tpajero, @lpica, @roneil, and @fsouzade

changed target branch from master to charm-thor-hlt2-lines

added 1 commit

• aa29ff6e - [charm/detection_asymmetry_lines] update line and combiner names, add functors...

Compare with previous version

resolved all threads

added 1 commit

• ef04469e - Mass cut was invalid

Compare with previous version

added 1 commit

• b335111a - remove duplicated cut; add charges to decay descriptors in docstrings

Compare with previous version

Hi @ldufour!

Thanks for preparing these lines. Some general comments:

1. Will the lines be used only in charm measurements, or also by other WGs? In the second case, I think their rate shouldn't be attributed to the Charm WG and they should go in another dedicated folder
2. I don't see any requirements on the presence of pvs nor the general event cut. Is this an oversight or a desired feature? Or maybe will it be added with the HLT1 filtering?
3. You call make_pvs in all the instances of your lines. Are there any reasons not to declare them as defaults in the called functions instead?
4. @mstahl, the line LcpToKsPp should now be LcpToPpKs according to the rules here. Or maybe we can just warn about this second exception to the rule in the docs

added 1 commit

• 2438710e - remove duplicated cut; add charges to decay descriptors in docstrings

Compare with previous version

Hi @ldufour, we are still trying to merge remaining charm-related hlt2 lines, even if we are past the deadline. Are you still intended to have this merged?

added 368 commits

• 2438710e...1aad3305 - 368 commits from branch charm-thor-hlt2-lines

Compare with previous version

changed target branch from charm-thor-hlt2-lines to master

added 1 commit

• 62ef4af2 - Changes

Compare with previous version

added 1 commit

• 417948b4 - Changes

Compare with previous version

added 10 commits

• 417948b4...a84abc21 - 8 commits from branch master
• 92713847 - use in_range; remove unnecessary (())
• 6589820f - Merge branch 'master' into ldufour-charm-hlt2-asymmetry-lines

Compare with previous version

approved this merge request

assigned to @ldufour

added 1 commit

• 6d654ec3 - add GEC require_pvs prefilters to all lines

Compare with previous version

approved this merge request

resolved all threads

Hi @thboettc, this is ready for testing