PerfMonComps: Refactor the PerfMonMT measurements/data and clean things up a little bit
Although there seems to be a lot of changes in this MR the core functionality shouldn't actually change. We split the single Measurement
/MeasurementData
into dedicated structs. We now have two measurements: ComponentMeasurement
and SnapshotMeasurement
, and three data structs to hold these: ComponentData
, EventLevelData
, and SnapshotData
. This way, we can unify the interface a little bit (i.e. no ad-hoc fnc_component
, fnc_snapshot
etc. anymore). While at it, I also tried to unify the naming conventions a bit but that's not completely done yet.
Additionally, we now also store the component-level wall-time measurements in the JSON that were previously excluded.
Merge request reports
Activity
This merge request affects 1 package:
- Control/PerformanceMonitoring/PerfMonComps
This merge request affects 3 files:
- Control/PerformanceMonitoring/PerfMonComps/src/PerfMonMTSvc.cxx
- Control/PerformanceMonitoring/PerfMonComps/src/PerfMonMTSvc.h
- Control/PerformanceMonitoring/PerfMonComps/src/PerfMonMTUtils.h
added Core master review-pending-level-1 labels
CI Result SUCCESS (hash 98770526)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon 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
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 40963]added review-approved label and removed review-pending-level-1 label
mentioned in commit d9432939
added sweep:ignore label