Pierre-Alain Loizeau authored 1 year ago and Pierre-Alain Loizeau committed 1 year ago

- Add optional pick starting decade and starting 0 in GenerateLogBinArray histo tool
- Reduce logs from TS source and Unpack task

- Prevent FairrootManager from touching the source object trhogh its pointer after the run is over
- Explicitely delete the dynamically allocated Run and Source objects in the macro (give up share ownership of config objects)
- Give up ownership of the config objects, which triggers their destruction before reaching end of the macro (last owner of shared pointer)

Problem was that the FairrootManager was never going out of scope and was not calling the Source destructor.
So all shared pointers and the source objects were destroyed only when going out of scope in the ROOT session, at which time ROOT already deleted some TF1's, THx's and Canvases without asking anybody (global Lists).
The source and config destructors were then calling destructors for their members and member's members, where we tried to cleanup the memory we assigned but which ROOT already deleted.

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

- Add vector constructor to CbmSourceTsArchive
- Call string constructor of FLES IPC TimesliceAutoSource if vector of size 1
- Make vector input default in run_unpack macro
- Add alternative call with single string to run_unpack_macro to keep legacy interface available

Jan de Cuveland authored 3 years ago and Florian Uhlig committed 3 years ago

This change corresponds to Pascal's request at
!455 (comment 13399)

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.

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

Since, the FairSource::Source_Type setting changes the behavior on the reading of the first Timeslice, its usage is for the time being removed. Therefore, a CbmSourceType enum is introduced which handles now the switch between data reading from online and offline sources.

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.