Gitlab CI authored 2 years ago and Gerhard Raven committed 2 years ago

patch generated by https://gitlab.cern.ch/lhcb/LHCb/-/jobs/22520194

... instead, just create a single map of locationID -> buffer for that
locationID. Update the unpackers to use the map as input and request the
relevant buffer from it instead of looking for them directly on the TES.

This eliminates the 'packed' locations and replaces all of them with a
single TES location with a map in it.

o used for mapping TES locations to types -- but this mapping is
  not required at _runtime_, but during configuration time so no
  need to push this through the configurable corresponding to a
  service...

and pass file manifest explicilty instead of hijacking the HltANNSvc configurable as global state

* improve diagnostic if encoding table does not contain expected entry
* support lookup by oid in GitANNSvc

- no longer use the 5 most significant bits of the source id.
  => As a result all source id encodings have slightly changed

- DstData:
    move compression from the sourceID into the payload,
    add the encoding key used into the payload

- HltDecReports
    add the encoding key to the payload

- HltSelReports
    no changes to the payload

... and move code in HltDAQ into the DAQ namespace (instead of PackedData)

- and propagate it when merging/splitting buffers
- and use it during packing/unpacking

- replace HltRawBankDecoderBase base class with a service and
  standalone utility funtions
- update decoder classes accordingly
- update the IIndexedANNSvc interface so that it can be used instead
  of the HltANNSvc to implement all int->string mappings.
  (as the interface is a 'versioned superset' of the HltANNSvc)
- update encoder and decoder classes accordingly
  NOTE: this included the ExecutionsReportWriter!
- replace use of non-thread-safe Error and Warning with MsgCounter

HltTrackingCoder

Lbexec improvements

See merge request !3725

Remove getDet in CaloFutureElectron

See merge request !3741

Remove getDet in CaloFuture2DView

See merge request !3717

Sort LHCbIDs before returning std::vector<LHCb::LHCbID>

See merge request !3746

Changed calculateChannelAndFrac to stop passing by reference the fraction

See merge request !3745

Change default raw_event_format to 0.5

See merge request !3687

DeRichPMTPanel: protect against null test PD pointer

See merge request !3743

Protect DeRichPMT::readPMTPropertiesFromDB from NULL condition

See merge request !3737

Check SciFi banks and sort them by hand

See merge request !3735