Adding counters to store scifi hits

Currently the list of initialisations is hardcoded, looking for something more elegant.

Gives something like

PrStoreSciFiHits                       INFO Number of counters : 25
 |    Counter                                      |     #     |    sum     | mean/eff^* | rms/err^*  |     min     |     max     |
 | "Average X in T1U"                              |     36268 |   -1071582 |    -29.546 |     1265.5 |     -2656.4 |      2656.2 |
 | "Average X in T1V"                              |     36185 |   -1440091 |    -39.798 |     1272.6 |     -2656.4 |      2656.1 |
 | "Average X in T1X1"                             |     36092 |  -882685.5 |    -24.457 |     1270.6 |     -2645.9 |      2645.9 |
 | "Average X in T1X2"                             |     36654 |  -854720.3 |    -23.319 |     1251.3 |     -2646.1 |      2646.1 |
 | "Average X in T2U"                              |     34951 |  -310020.3 |    -8.8701 |     1274.3 |     -2656.0 |      2656.3 |
 | "Average X in T2V"                              |     36259 |  -938162.6 |    -25.874 |     1265.8 |     -2656.2 |      2656.2 |
 | "Average X in T2X1"                             |     33170 |    -187509 |    -5.6530 |     1260.3 |     -2646.0 |      2645.6 |
 | "Average X in T2X2"                             |     37180 |  -452010.1 |    -12.157 |     1247.5 |     -2646.1 |      2645.9 |
 | "Average X in T3U"                              |     38680 |   112486.5 |     2.9081 |     1514.4 |     -3188.0 |      3187.7 |
 | "Average X in T3V"                              |     39549 |   99427.91 |     2.5140 |     1505.6 |     -3188.3 |      3188.1 |
 | "Average X in T3X1"                             |     38024 |   292939.9 |     7.7041 |     1519.5 |     -3176.1 |      3175.9 |
 | "Average X in T3X2"                             |     40570 |  -435481.2 |    -10.734 |     1502.4 |     -3176.2 |      3176.2 |
 | "Hits in T1U"                                   |       400 |      36268 |     90.670 |     40.150 |      23.000 |      291.00 |
 | "Hits in T1V"                                   |       400 |      36185 |     90.463 |     41.070 |      23.000 |      260.00 |
 | "Hits in T1X1"                                  |       400 |      36092 |     90.230 |     39.217 |      26.000 |      287.00 |
 | "Hits in T1X2"                                  |       400 |      36654 |     91.635 |     40.085 |      25.000 |      281.00 |
 | "Hits in T2U"                                   |       400 |      34951 |     87.377 |     39.309 |      27.000 |      252.00 |
 | "Hits in T2V"                                   |       400 |      36259 |     90.647 |     40.558 |      20.000 |      254.00 |
 | "Hits in T2X1"                                  |       400 |      33170 |     82.925 |     37.373 |      18.000 |      245.00 |
 | "Hits in T2X2"                                  |       400 |      37180 |     92.950 |     41.939 |      30.000 |      256.00 |
 | "Hits in T3U"                                   |       400 |      38680 |     96.700 |     41.093 |      33.000 |      256.00 |
 | "Hits in T3V"                                   |       400 |      39549 |     98.873 |     42.241 |      28.000 |      292.00 |
 | "Hits in T3X1"                                  |       400 |      38024 |     95.060 |     39.492 |      31.000 |      243.00 |
 | "Hits in T3X2"                                  |       400 |      40570 |     101.42 |     42.194 |      26.000 |      258.00 |
 | "Total number of hits"                          |       100 |     443582 |     4435.8 |     1810.6 |      1602.0 |      11106.0 |

Technically the Average X counter contains the info of the Hits counters

added RTA label

changed the description

added 1 commit

• e89eebf7 - Changed to a stat counter

Compare with previous version

I'm not sure we can be much more elegant without sacrifying readability. I think we can make it work with variadic templating (to be proven still) but not worth it here. Can you just fix the pipeline (== apply the formatting) ? then I test and merge.

added 1 commit

• 54141562 - Changed a bit the types of counters in the seeding

Compare with previous version

added 1 commit

• fdf7a424 - Fixed formatting

Compare with previous version

added needs ref update label

marked this merge request as ready

changed the description

added 17 commits

• fdf7a424...b8715381 - 13 commits from branch master
• cdbe4c73 - Adding counters to store scifi hits
• d0a19041 - Changed to a stat counter
• b30bce72 - Rebase on master
• 859fcaff - Fixed formatting

Compare with previous version

resolved all threads

mentioned in issue Moore#426 (closed)

/ci-test

added ci-test-triggered label

• [2022-05-13 11:38] Validation started with lhcb-master-mr#4419
• [2022-05-14 21:40] Validation started with lhcb-master-mr#4432

mentioned in commit Moore@7911f168

mentioned in commit Alignment@b67a0a0e

mentioned in merge request Alignment!269 (merged)

mentioned in commit Panoptes@df19923e

The RefBot pipeline created the following reference update MRs: Moore!1558 (merged), Alignment!269 (merged), Panoptes!157 (merged)

Click this to see encountered Alignment warnings

Alignment/x86_64_v2-centos7-clang12-opt: Test Humboldt.iterator-vp-halves-modules contains nonzero exit code and/or warnings!
Alignment/x86_64_v2-centos7-clang12-opt: Test without ref failed: Humboldt.iterator-vp-halves-modules!

Click this to see encountered Moore warnings

Moore/x86_64_v2-centos7-gcc11-opt: Test RecoConf.hlt2_plot_calo_resolution_gamma was not run because of failed dependencies and is possibly missing a reference update!
Moore/x86_64_v2-centos7-gcc11-opt: Test RecoConf.hlt2_reco_baseline_multi_threaded was not run because of failed dependencies and is possibly missing a reference update!
Moore/x86_64_v2-centos7-gcc11-opt: Test without ref failed: Hlt1Conf.dump_for_standalone_allen!
Moore/x86_64_v2-centos7-gcc11-opt: Test without ref timed out: Hlt1Conf.persistency.allen_dst_read!
Moore/x86_64_v2-centos7-gcc11-opt: Test without ref timed out: Hlt1Conf.persistency.allen_secondary_vertex_tos!
Moore/x86_64_v2-centos7-gcc11-opt: Test without ref failed: Hlt2Conf.profile_config!
Moore/x86_64_v2-centos7-gcc11-opt: Test without ref timed out: Hlt2Conf.test_hlt2_trackefflines!
Moore/x86_64_v2-centos7-gcc11-opt: Test without ref timed out: Hlt2Conf.test_hlt2_noUT_trackefflines!
Moore/x86_64_v2-centos7-gcc11-dbg: Test Hlt1Conf.persistency.allen_dst_read was not run because of failed dependencies and is possibly missing a reference update!
Moore/x86_64_v2-centos7-gcc11-dbg: Test RecoConf.hlt2_plot_calo_resolution_gamma was not run because of failed dependencies and is possibly missing a reference update!
Moore/x86_64_v2-centos7-gcc11-dbg: Test RecoConf.hlt2_reco_baseline_multi_threaded was not run because of failed dependencies and is possibly missing a reference update!
Moore/x86_64_v2-centos7-gcc11-dbg: Test without ref timed out: Hlt1Conf.persistency.allen_dst_write!
Moore/x86_64_v2-centos7-gcc11-dbg: Test without ref failed: Hlt1Conf.dump_for_standalone_allen!
Moore/x86_64_v2-centos7-clang12-opt: Test RecoConf.hlt2_plot_calo_resolution_gamma was not run because of failed dependencies and is possibly missing a reference update!
Moore/x86_64_v2-centos7-clang12-opt: Test RecoConf.hlt2_reco_baseline_multi_threaded was not run because of failed dependencies and is possibly missing a reference update!
Moore/x86_64_v2-centos7-clang12-opt: Test without ref failed: Hlt1Conf.dump_for_standalone_allen!
Moore/x86_64_v3-centos7-gcc11-opt+g: Test RecoConf.hlt2_plot_calo_resolution_gamma was not run because of failed dependencies and is possibly missing a reference update!
Moore/x86_64_v3-centos7-gcc11-opt+g: Test RecoConf.hlt2_reco_baseline_multi_threaded was not run because of failed dependencies and is possibly missing a reference update!
Moore/x86_64_v3-centos7-gcc11-opt+g: Test without ref failed: Hlt1Conf.dump_for_standalone_allen!

mentioned in merge request Moore!1558 (merged)

mentioned in merge request Panoptes!157 (merged)

approved this merge request

resolved all threads

merged

mentioned in commit fcbdd25b

mentioned in commit Moore@dd5b0949

mentioned in commit Alignment@cc14f2ef

mentioned in issue Moore#432 (closed)

mentioned in commit Moore@04df2527