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.

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 Pierre-Alain Loizeau committed 3 years ago

Since, we do not use the class name in our output branches I added a GetBranchName() function for all digis connected to the mcbm2021 beamtime. @todo add the function to other digis (much,mvd).

After the long standing problem with the external package AnalysisTree was
fixed the tests with the latest FairSoft/FairRoot versions are enabled.
Test on Debian 8, Debian 10 and CentOS 7.

This version fixes a long standing problem of AnalysisTree with
ROOT versions >= 6.20. The produced files couldn't be read anylonger.
The problems is discussed in detail at https://redmine.cbm.gsi.de/issues/2108

- remove the tof_disim test and in fact the full macro/tof/CMakeLists.txt
- move macro/tof/digitizer/GenTofBdfRootInput_v17a.C one level up
- remove the macro/tof/digitizer folder
- remove the macro/tof/qa folder

drslebedev authored 3 years ago and Florian Uhlig committed 3 years ago

Adding lmvm combinatorial pairs procedure

Develop combinatorial BG procedure and add corresponding histograms

Correct pairing in CB and reverse formatting of python macros

Add Single Particle Yield Histos

Particle Yield and Acceptance of Electrons and Positrons is observed for various detector combinations.
Formatting of python scripts after they had been tored by wrong application of clang format.

WIP Add single particle histos

WIP: Update single particle histograms

WIP: Add histograms for nPoint

Number of points the electrons / positrons leave in the various detectors

Modify macros for common production

Also some small adaptions in CbmAnaDielectronTask.cxx and in ana and litqa macros were done.

Apply clang format

If the ROOT session is closed after the macro has finished the TGeoManger
has to be properly destructed.
This is done calling the macro RemoveGeoManager.
Add thecall in macro where it was missing.