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.

The machine providing the CbmRoot_macosx gitlab-runner is currently broken.

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

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.

Adrian A. Weber authored 3 years ago and Pierre-Alain Loizeau committed 3 years ago

add mRICH time offset correction and add masking of DiRICH boards to avoid the error messages. Error messages tells you now, which board has a problem.

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

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

Add copy assignment operator definition for CbmTrdDigi

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

Fix some warnings for Psd Rich and Tof algorithms

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

Implemented empty variables for configs. Such that an if based add can be used for add to source and the part below "need not to be touched" also does not need to be touched to (de)activate configs.

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

The emplace_back call was missing, which lead to the fact that no digis could be found.
- Remark time extraction to be checked by experts no correlations visible to the Trd in +- 1000 ns

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

Had to include some directorie in the L1 CmakeList

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

Removed the setDerivedTsParameters empty function implementation. It is handled by the base template
To be reimplemented when needed.

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.