HistogramSpec: add missing L types #13427
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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!