[hist] Implement initial RProfile#22588
Conversation
| \warning This is part of the %ROOT 7 prototype! It will change without notice. It might trigger earthquakes. | ||
| Feedback is welcome! | ||
| */ | ||
| class RProfile final { |
There was a problem hiding this comment.
Imho this should be called something more specific to histograms (RHistProfile? RProfileHist?), as "profile" is a generic term that could be overloaded (e.g. a timing profile) and since it's directly in the ROOT namespace we might want to avoid that confusion.
There was a problem hiding this comment.
hm, I think the term "profile" is well established in HEP to not be confusing in the context of a data analysis framework. For example, historically the class is called TProfile, Boost.Histogram calls it make_profile, and for YODA it's BinnedProfile / Profile1D etc.
There was a problem hiding this comment.
I'm more concerned about ambiguities in internal usage by developers than about confusion from users (RProfile might be used as an internal class in some other context). Nothing that some namespaces can't solve of course, but might be worth thinking about.
Test Results 20 files 20 suites 3d 4h 10m 6s ⏱️ Results for commit ce6f93e. ♻️ This comment has been updated with latest results. |
It builds on RHistEngine with an additional variable.
SetBinContent, Add, AddAtomic, Clear, Clone, Scale; implemented similarly to RHist, forwarding to RHistEngine and RHistStats.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #22588 +/- ##
==========================================
+ Coverage 59.75% 59.77% +0.02%
==========================================
Files 3367 3368 +1
Lines 494970 495515 +545
Branches 95206 95270 +64
==========================================
+ Hits 295749 296187 +438
- Misses 199221 199328 +107
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
It builds on
RHistEnginewith an additional variable.