add TTree support for DQ algorithms
Addresses ATLASDQ-665 (adds TTree support to AthenaMonitoringKernel). WIP until !29214 (merged) is merged as there are a couple of minor tweaks to adjust to the updated API.
User interface specifies TTree branches using the new "treeDef" argument of the new defineTree. All the standard TTree primitive types (except for C = C null-terminated string) are supported, as is "string" and a number of vector types (float, double, char, int, long, and the unsigned versions of the integer types).
Cherry-picks a commit from !29191 (merged) (for proper MT operation).
Merge request reports
Activity
- Resolved by Peter Onyisi
Glad to see it actually. It may be way more efficient sometimes to produce trees than the histograms. Online, though we do not have transport layer for that kind of data. I wonder if we ever had any use-case for which it would be helpful. Re-tagging @fwinkl on this.
added 676 commits
-
6e0b5176...914cf50a - 674 commits from branch
atlas:master
- 4a2bd106 - Merge branch 'master' of https://gitlab.cern.ch:8443/atlas/athena into master-dq-add-ttree-support
- f75f7e07 - Respond to review comments
-
6e0b5176...914cf50a - 674 commits from branch
This merge request affects 2 packages:
- Control/AthenaMonitoring
- Control/AthenaMonitoringKernel
Adding @ssnyder as watcher
added Core DQ master review-pending-level-1 labels
CI Result FAILURE (hash f75f7e07)Athena AthSimulation externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 8409]added review-user-action-required label and removed review-pending-level-1 label
added 452 commits
-
9e4e9b8a...2ddcee1e - 450 commits from branch
atlas:master
- 91461ef5 - Merge branch 'master' of https://gitlab.cern.ch:8443/atlas/athena into master-dq-add-ttree-support
- 2b5c37b3 - Resolve merge conflict
-
9e4e9b8a...2ddcee1e - 450 commits from branch
This merge request affects 2 packages:
- Control/AthenaMonitoring
- Control/AthenaMonitoringKernel
Adding @ssnyder as watcher
added review-pending-level-1 label and removed review-user-action-required label
CI Result SUCCESS (hash 2b5c37b3)Athena AthSimulation externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 8575]added review-approved label and removed review-pending-level-1 label
mentioned in commit 9eb9826f
added sweep:ignore label