LArHVData: put current and hv into the same obj, saves memory
Inspired by the discussion of ATEAM-726.
Simplify the layout of LArHVData, saves some 30MBytes vmem on q431.
Merge request reports
Activity
This merge request affects 2 packages:
- LArCalorimeter/LArRecConditions
- LArCalorimeter/LArRecUtils
Affected files list will not be printed in this case
Adding @pavol as watcher
added JetEtmiss LAr master review-pending-level-1 labels
Running on 1000 events (serial, no MP or MT) from
data18_13TeV.00355877.physics_Main.merge.DRAW_ZMUMU.f950_m1831._0041.1
, vmem goes from 8251 to 7732 MBytes. This job goes through 109 IOVs of the LAr HV DCS data.For an MP job, the memory consumption due to conditions objects is multiplies by the number of workers, they are not shared.
The AOD file is identical.
- Walter
CI Result SUCCESS (hash dbc08e61)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 30924]removed review-pending-level-1 label
added review-approved label
mentioned in commit 8bf4352e
added sweep:ignore label
mentioned in merge request !42309 (merged)