H5 Writer: allow double -> half

We write half precision a lot, but there wasn't a good way to convert directly from a double to a half precision output. Instead we had to do some double -> float -> half magic. This allows direct conversion from double -> half.

Also added a unit test for short int and double -> half.

This merge request affects 1 package:

• PhysicsAnalysis/AnalysisCommon/HDF5Utils

This merge request affects 3 files:

• PhysicsAnalysis/AnalysisCommon/HDF5Utils/HDF5Utils/CompressedTypes.h
• PhysicsAnalysis/AnalysisCommon/HDF5Utils/Root/CompressedTypes.cxx
• PhysicsAnalysis/AnalysisCommon/HDF5Utils/util/test-hdf5-writer.cxx

Could not map the following files to a domain:

• PhysicsAnalysis/AnalysisCommon/HDF5Utils/HDF5Utils/CompressedTypes.h
• PhysicsAnalysis/AnalysisCommon/HDF5Utils/util/test-hdf5-writer.cxx
• PhysicsAnalysis/AnalysisCommon/HDF5Utils/Root/CompressedTypes.cxx

Adding @dguest as watcher

added analysis-review-required main no-domain review-pending-level-1 labels

CI Result SUCCESS (hash 1b4b6193)

	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-CC7 79443]

Approved for analysis release purposes.

Nils [Analysis RC]

added analysis-review-approved label and removed analysis-review-required label

Looks good to me, approving.

Cheers, Philipp (L1)

removed review-pending-level-1 label

added review-approved label

merged

mentioned in commit ca77da74

added sweep:ignore label