Prevent usage of histograms with wrong number of coordinates
All threads resolved!
All threads resolved!
One used to be able to use e.g. a 2D histogram with a syntac like ++histo[singleNumber]
The second and missing number wa simply set to 0.
This is now creating a compilation error and a test was added to check that error
Closes #226 (closed)
Edited by Sebastien Ponce
Merge request reports
Activity
- Resolved by Marco Clemencic
@clemenci 2 small notes :
- I'm not fan of the long set of lines in the CMakefile for the new test. Do you see a way to reduce that ?
- the pre-commit hook of Gaudi is failing (hanging forever) for me since yesterday. Anything new there ? I had to deactivate it for this one
added 1 commit
- 6d921841 - Prevent usage of histograms with wrong number of coordinates
added 1 commit
- 806e8a61 - Prevent usage of histograms with wrong number of coordinates
changed milestone to %v36r6
assigned to @clemenci
added lhcb-gaudi-head label
- [2022-06-18 00:04] Validation started with lhcb-gaudi-head#3268
- [2022-06-18 00:42] Validation started with lhcb-run2-gaudi-head#340
- [2022-06-19 00:05] Validation started with lhcb-gaudi-head#3269
- [2022-06-21 00:03] Validation started with lhcb-gaudi-head#3270
- [2022-06-22 00:05] Validation started with lhcb-gaudi-head#3271
- [2022-06-22 00:49] Validation started with lhcb-run2-gaudi-head#341
- [2022-06-23 00:04] Validation started with lhcb-gaudi-head#3272
- [2022-06-24 00:03] Validation started with lhcb-gaudi-head#3273
- [2022-06-25 00:06] Validation started with lhcb-gaudi-head#3274
- [2022-06-25 00:52] Validation started with lhcb-run2-gaudi-head#342
- [2022-06-26 00:08] Validation started with lhcb-gaudi-head#3275
- [2022-06-26 00:35] Validation started with lhcb-run2-gaudi-head#343
- [2022-06-27 00:03] Validation started with lhcb-gaudi-head#3276
- [2022-06-28 00:04] Validation started with lhcb-gaudi-head#3277
- [2022-06-29 00:04] Validation started with lhcb-gaudi-head#3278
- [2022-06-29 00:43] Validation started with lhcb-run2-gaudi-head#344
- [2022-06-30 00:04] Validation started with lhcb-gaudi-head#3279
- [2022-06-30 00:44] Validation started with lhcb-run2-gaudi-head#345
Edited by Software for LHCbmentioned in merge request lhcb/Online!730 (merged)
mentioned in commit 3a0d864e
mentioned in merge request atlas/athena!55604 (closed)
Please register or sign in to reply