Streamline LumiFSRtoTTree

• more paranoid dynamic_cast
• keep count per run from the start instead of keeping all individual counts and summing at the end
• avoid std::list
• remove unnecessary includes
• make explicitly thread-safe

requested review from @egraveri

added 1 commit

• 36f9cc09 - Streamline LumiFSRtoTTree

Compare with previous version

changed the description

added RTA label

Thank you @graven. For me all is fine, can you (or @efranzos) just run the lumi-related tests before merging?

make fast/LHCb/test ARGS='-R FileSummaryRecord.read_lumi_to_tree'
make fast/DaVinci/test ARGS='-R DaVinciExamples.tupling.test_davinci_tupling_with_lumi_from_spruce' 

added DPA-WP1 Luminosity labels

requested review from @efranzos

assigned to @mzdybal

/ci-test

added ci-test-triggered label

• [2023-10-09 14:52] Validation started with lhcb-master-mr#9407
• [2023-10-10 10:30] Validation started with lhcb-master-mr#9414
• [2023-10-27 13:50] Validation started with lhcb-master-mr#9560
• [2023-10-28 16:09] Validation started with lhcb-master-mr#9564

added 1 commit

• 17372f0c - make LumiFSRtoTTree explicitly thread-safe

Compare with previous version

added 1 commit

• ad64a9db - make LumiFSRtoTTree explicitly thread-safe

Compare with previous version

added thread safety label

after a small discussion with @clemenci, decided that it would be better to make the code explicitly thread-safe -- so I added an extra commit to this MR

/ci-test

changed the description

mentioned in issue Moore#665 (closed)

Assigning to new shifter @jagoodin .

assigned to @jagoodin and unassigned @mzdybal