Update tagging array qmtest including mctruth infos

Update the test_davinci_tupling_array_taggers qmtest adding a few lines showing how to store the mctruth information related to the extra particles in the event, beside the ones in the decay chain. In order to retrieve correctly the mctruth information, the input file has changed and the reference has been updated accordingly.

Two default values has to be defined in order to ensure a valid result if the functor's output is empty:

• in case of functors returning an integer/bool or an array of integers/bools, as TRUEID or TagTr_TRUEID[nTags];
• inside the MAP_INPUT_ARRAY when the inner functor is defined via lambda function, as MCTRUTH_pions.
  For example:

        "TagTr_TRUEID[nTags]":
        F.VALUE_OR([0]) @ F.MAP_INPUT_ARRAY(
            Functor=F.VALUE_OR(0) @ MCTRUTH_pions(F.PARTICLE_ID),
            Relations=tagAlg_rels),

Needs Rec!3139 (merged).

added DPA-WP3 testing labels

requested review from @pkoppenb and @erodrigu

assigned to @dfazzini

added 2 commits

• 2bf4e792 - Fixed formatting
• 30669c44 - fix typo

Compare with previous version

added 1 commit

• 40182925 - use nan value from Functor module

Compare with previous version

changed the description

@pkoppenb I think this MR is ready to be tested, it's a simple update of a single qmtest.

You mention in the header the bug reported by @amathad in issue Rec#385 (closed). It would have been ideal to have the latter sorted to avoid temporary workarounds. Can you 2 folks press RTA WP3 coordinators to get issue Rec#385 (closed) sorted asap? In general framework bugs should be prioritised in my opinion. Please @mvesteri and @poluekt can you look and act? Many thanks.

Morning @dfazzini. Very nice and useful update this is. I posted a couple of trivial suggestions. I am already approving as I'm asking for doc. Feel free to trigger the CI when ready since Patrick is rather unavailable today.

approved this merge request

added 1 commit

• 0e0a48b1 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

mentioned in issue Rec#385 (closed)

@dfazzini : I hope this MR Rec!3139 (merged) fixes the issue. Can you please introduce a dependency on that and check if it works?

mentioned in merge request Rec!3139 (merged)

changed the description

added 1 commit

• 4ba7abf0 - small fix

Compare with previous version

added 1 commit

• 03924ec9 - add comment about definition of default values

Compare with previous version

added 1 commit

• 7d9b07fb - Fixed formatting

Compare with previous version