New LArCaliWave persistent version

This should solve ATLLARONL-993

For SuperCell calibration data processing we need to store DAC in more then 16 bits. Therefore additional member for pulsedLine is added and DAC in is used in full 32 bits. Changed also TP converters and added a test for this new version.

Code is used only in LAr calibration data processing, not in standard reconstruction.

added bugfix label

This merge request affects 5 packages:

• LArCalorimeter/LArCalibTools
• LArCalorimeter/LArCalibUtils
• LArCalorimeter/LArCnv/LArCondTPCnv
• LArCalorimeter/LArExample/LArCalibProcessing
• LArCalorimeter/LArRawConditions

Affected files list will not be printed in this case

Adding @pavol as watcher

added JetEtmiss LAr master review-pending-level-1 labels

CI Result SUCCESS (hash 4d61691b)

	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 33640]

Looks good. There seem to be a lot of leftover cout statements (presumably from debugging). It would be great if these could be tidied before this is approved but I won't insist if it need to go in urgently. Tom - L1 shifter

added review-user-action-required label and removed review-pending-level-1 label

added 1 commit

• 38453104 - address the shifter comments

Compare with previous version

resolved all threads

This merge request affects 5 packages:

• LArCalorimeter/LArCalibTools
• LArCalorimeter/LArCalibUtils
• LArCalorimeter/LArCnv/LArCondTPCnv
• LArCalorimeter/LArExample/LArCalibProcessing
• LArCalorimeter/LArRawConditions

Affected files list will not be printed in this case

Adding @pavol as watcher

added review-pending-level-1 label and removed review-user-action-required label

CI Result SUCCESS (hash 38453104)

	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 33916]

added review-approved label and removed review-pending-level-1 label

mentioned in commit bee2307f

merged

added sweep:ignore label

mentioned in merge request !58562 (merged)