Pierre-Alain Loizeau authored 3 years ago and Volker Friese committed 3 years ago

- Add Binning FW support to the Print method of the StsXyterMessage, enabled by default
- Fix the duplicate digis filtering in the STS unpacker and make it flag controllable
- Add an optional full time sorting to the Unpackers steering class
- Related mCBM macros changes

Before this fix only the number of digis of the last component of a subsystem endet up in the counter of the event header. Now an Add function is used and the counters a resetted at the beginning of each timeslice.

Pascal Raisig authored 3 years ago and Florian Uhlig committed 3 years ago

This commit adds the measurement of the cpu and wall time to the unpacking algo template/CbmRecoUnpack::unpack function.
The values are printed add the end of the run and stored additionally in an output histogram.

It was not checked if a Trd1D config exists before trying to get its main branch name. This caused a seg violation in case no Trd1D config was added. Now the safeties are adjusted to check for it.
Includes a minor update of the naming scheme

Pascal Raisig authored 3 years ago and Pierre-Alain Loizeau committed 3 years ago

Remove by hand added CbmTsEventHeader from CbmRecoUnpack. Instead, use the getter from the FairRun::Instance() to access the event header. Remark a fatal will be created if the wrong event header is set in the steering macro

Pierre-Alain Loizeau authored 3 years ago and Pierre-Alain Loizeau committed 3 years ago

Switch form info to log for debug log statement

Pierre-Alain Loizeau authored 3 years ago and Pierre-Alain Loizeau committed 3 years ago

Changed severity for large amount of LOG statements

Changed default from info to debug for a larger amount of tof log messages. Otherwise, it makes info reading with all subsystems almost impossible.

Pascal Raisig authored 3 years ago and Pierre-Alain Loizeau committed 3 years ago

Unfortunately this does not yet produce digis, but I am note sure, if it is an parameter based issue or an issue with the implementation. I think it needs some experts from the Sts to have a look. Which is why I am pushing it.

Pascal Raisig authored 3 years ago and Pierre-Alain Loizeau committed 3 years ago

For the current situation with the TRD1D and TRD2D configs trying to create and access in principle the same "TrdDigi" branch a small workaround in the CbmRecoUnpack was required. Such that the first Config does not loose the correct pointer when the seconds overwrites the branch in the FairRootManager.

Pascal Raisig authored 3 years ago and Pierre-Alain Loizeau committed 3 years ago

Seems to have similar issues with the includes like PSD.

Pascal Raisig authored 3 years ago and Pierre-Alain Loizeau committed 3 years ago

Except for Psd the classes are empty dummies.

Pascal Raisig authored 3 years ago and Volker Friese committed 3 years ago

Integration of TRD unpacking, with empty implementations for unpacker, monitor container and parameter container.