Correctly group histograms in ROOT files

The previous version was using <component>/<hist_id> as histogram id in the ROOT file, but it is better to create a directory called <component> and store the object <hist_id> in it. If the histogram id contains /s, the corresponding directories are created too.

With !1234 (63917b20) histograms of tools are stored in subdirectories of the parent. For example, the histogram MyHist of the tool MyAlg.MyTool is stored in a ROOT file with the path MyAlg/MyTool/MyHist.

To-do:

• add test

changed milestone to %v36r1

added lhcb-gaudi-head label

• [2021-07-14 00:06] Validation started with lhcb-gaudi-head#2986
• [2021-07-14 00:54] Validation started with lhcb-run2-gaudi-head#166
• [2021-07-15 00:06] Validation started with lhcb-gaudi-head#2987
• [2021-07-15 01:28] Validation started with lhcb-run2-gaudi-head#167
• [2021-07-16 00:04] Validation started with lhcb-gaudi-head#2988
• [2021-07-16 00:44] Validation started with lhcb-run2-gaudi-head#168
• [2021-07-17 00:30] Validation started with lhcb-gaudi-head#2989
• [2021-07-17 01:23] Validation started with lhcb-run2-gaudi-head#169
• [2021-07-18 00:03] Validation started with lhcb-gaudi-head#2990
• [2021-07-18 00:10] Validation started with lhcb-run2-gaudi-head#170
• [2021-07-20 00:03] Validation started with lhcb-gaudi-head#2991
• [2021-07-20 00:31] Validation started with lhcb-run2-gaudi-head#171
• [2021-07-21 00:28] Validation started with lhcb-gaudi-head#2992
• [2021-07-21 01:29] Validation started with lhcb-run2-gaudi-head#172
• [2021-07-22 00:15] Validation started with lhcb-gaudi-head#2993
• [2021-07-22 01:08] Validation started with lhcb-run2-gaudi-head#173
• [2021-07-23 00:04] Validation started with lhcb-gaudi-head#2994
• [2021-07-24 00:15] Validation started with lhcb-gaudi-head#2995
• [2021-07-24 01:21] Validation started with lhcb-run2-gaudi-head#174

I just realized that this only takes into account the owner (algorithm) name, but not if the histograms are created with /s in the id to have internal grouping.

marked this merge request as draft

changed the description

mentioned in merge request !1235 (merged)

added 1 commit

• 6ae726fd - Take into account grouping of histograms in directories

Compare with previous version

added 2 commits

• db5709d9 - Test histogram writing and grouping for tools too
• 63917b20 - Store tools' histograms in subdirs of the parent

Compare with previous version

marked the checklist item add test as completed

marked this merge request as ready

changed the description

mentioned in merge request !1236 (merged)

approved this merge request

added 1 commit

• 32a017ab - Refactor to move TDirectory management inside saveRootHistoInternal

Compare with previous version

added 1 commit

• 85b5c471 - User ranges::accumulate instead of a for loop

Compare with previous version

resolved all threads

approved this merge request

resolved all threads

merged

mentioned in commit 1853543b