Unify writing of data from Hlt into RawBank
The following discussion from !561 (merged) should be addressed:
-
@graven started a discussion: Not for this MR, but to start an 'issue': the writing to
RawBank
should be done uniformly and consistent with the otherPackedData
that is written out of the Hlt. This implies the data should be written with the right banktype, and the datastream should contain sufficient information to identify the payload (and thus provide the required provenance that a decoder in the future an be sure to interpret the data properly). So this has to hook into the code provided inEvent/EventPacker
, and not just done 'ad-hoc'. And the burden of adding this is not very large: one has to define an 'aggregate' (i.e. plain struct) which 'names' the data members, a and provides a version number and boiler-plate 'load' and 'store' functions (and in future hopefully even less).