New PID Substitution DaVinci Tool

Start from #305 (closed), implementing the comment #305 (comment 5363235).

This MR will add a new PID substitution Tool and a new DTF Tool for DaVinci, which has integrated the PID substitution. A minimal example of the usage could be:

DTF_KstG = DecayTreeFitter(
    Name = 'DTF_KstG',
    Input = B_Data,
    Substitutions = [
        ' B0 ->  ( K*(892)0 ->  K+ K-{{pi-}} ) gamma',
    ]
)
variables_all = FC({
    # Original particle
    'ORIGINAL_ID'      : F.PARTICLE_ID,
    'ORIGINAL_ENERGY'  : F.ENERGY,
    'ORIGINAL_CHI2DOF' : F.CHI2DOF,
    # DTF Bd2KstGamma
    'DTF_KSTG_ID'      : F.VALUE_OR(-1) @ DTF_KstG.get_info( F.PARTICLE_ID ),
    'DTF_KSTG_ENERGY'  : DTF_KstG.get_info( F.ENERGY      ),
    'DTF_KSTG_CHI2DOF' : DTF_KstG.get_info( F.CHI2DOF     ),
})

A real example could be found in https://gitlab.cern.ch/lhcb/DaVinci/-/blob/jzhuo_SubsPID/DaVinciExamples/python/DaVinciExamples/tupling/option_davinci_tupling_DTF_substitutePID.py

Other example for SubstitutePID without DTF could be found in https://gitlab.cern.ch/lhcb/DaVinci/-/blob/jzhuo_SubsPID/DaVinciExamples/python/DaVinciExamples/tupling/option_davinci_tupling_substitutePID.py

This MR should be test with DaVinci!756 (merged)

FYI: @amathad @pkoppenb @erodrigu @fevolle

Closes #305 (closed) Closes #309 (closed)

added DPA-WP3 Tuples labels

assigned to @jzhuo

added RTA label

mentioned in merge request DaVinci!756 (merged)

changed the description

marked this merge request as draft

@pkoppenb and @erodrigu : This MR comes out of the design discussion and one of the options me and @jzhuo choose. I can present few slides in tomorrow DPA WP3 meeting on our behalf.

https://indico.cern.ch/event/1165474/

added 1 commit

• 79db119e - Add counter and comments

Compare with previous version

requested review from @amathad

added 1 commit

• 18c23121 - Fixed formatting

Compare with previous version

marked this merge request as ready

Can we give this a test already?

resolved all threads

requested review from @pkoppenb and @erodrigu

There was a presentation of this at the DPA-WP3 meeting : https://indico.cern.ch/event/1165474/#3-new-pid-substitution-davinci. Note that the syntax is now different from what's in the header of this issue.

added 3 commits

• ca4276e4 - Adding the new syntax support and documentation
• 3aae4615 - Documentation and code cleanup
• bbc28b51 - More documentation and verbose

Compare with previous version

resolved all threads

added 1 commit

• 3a42c8a2 - Fixed formatting

Compare with previous version

added 1 commit

• f2d58283 - Fix the typo

Compare with previous version

changed the description

mentioned in issue Moore#482 (closed)

mentioned in issue Moore#485 (closed)

mentioned in issue Moore#487 (closed)

what's the status here?

Hi, yes, I just back this week. I will update the DaVinci/Example today and move this MR asap.

changed the description

added 1 commit

• f472902c - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit

• a215d6ef - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit

• 3d42ab44 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

@jzhuo Are we ready for a new round of test ? I see one comment not yet answered.

This at least needs to be fixed before.