HistogramSpec: add missing L types#13427
Conversation
Please consider the following formatting changes to AliceO2Group#13411
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
This PR did not have any update in the last 30 days. Is it still needed? Unless further action in will be closed in 5 days. |
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
@ddobrigk could you also mark it as not draft, so the tests run? |
|
Any news on this one? |
|
Recreated (with a rebased branch) here: #14333 - let's see if we can manage to follow this up this time :-) |
This PR adds the missing TH1L, TH2L and TH3L types to the framework. Note that THnL sparses - the more costly options, incidentally - are already defined. The 64-bit longs are especially interesting when dealing with fast simulation, a context in which a massive amount of tracks will for sure be handled.
I'm actually having some trouble compiling this as the compiler says THxLs are not defined, which is puzzling: they are defined together in
TH1.hin ROOT. Maybe I am missing something else? @aalkin when you have a moment, can you help me out? (but please do not let me disturb your weekend - only whenever you manage). Many thanks!