ZDC: change bool storage type to char

The decoration centroidEventValid in container ZdcSums is currently stored with type bool. Boolean types are implicitly converted to char to save disk space. This is causes an error when executing algorithms ZdcRecRun3 and ZdcNtuple separately (i.e., when variables are stored on disk between executions):

AnalysisAlg         ERROR SG::ExcAuxTypeMismatch: Type mismatch for aux variable `::centroidEventValid' (330); old type is char new type is bool.  Old allocator type is std::allocator<char> new allocator type is std::allocator<bool>

This merge request changes the storage type of centroidEventValid from bool to char and solves this problem.

added 1 commit

• 0c3ca015 - change storage type of centroidEventValid from bool to char

Compare with previous version

marked this merge request as ready

This merge request affects 2 packages:

• ForwardDetectors/ZDC/ZdcAnalysis
• ForwardDetectors/ZDC/ZdcNtuple

Affected files list will not be printed in this case

added 24.0 ForwardDetectors analysis-review-required review-pending-level-1 labels

CI Result SUCCESS (hash 0c3ca015)

	Athena	AnalysisBase	AthAnalysis
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9289] (remote access info)

Changes are clear and CI succeeds, approving for L1.

removed review-pending-level-1 label

added review-approved label

added review-approved-point1 label

added review-approved-tier0 label

mentioned in commit 60da4735

merged

mentioned in merge request !71365 (merged)