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

The hit counter was resetted for each event, while
the number of clusters correponds to the full timeslice.
Now the printed hitCounterTs counts all hits in a TS without a reset.
No explicit hit counting per event is currently implemented.

Volker Friese authored 4 years ago and Florian Uhlig committed 4 years ago

Fix problem in CbmPVFinderKF (declaration of C array with non-positive size; redefine digi output file name for json to avoid overwriting conflict in the tests; slightly re-arrange tests from macro/run. Refs 2076.

Volker Friese authored 4 years ago and Florian Uhlig committed 4 years ago

Modified CbmTofSimpClusterizer such as to read from and write to CbmEvents in case of event-based reconstruction. Refs #2074.

The class FairLogger is  meanwhile only a wrapper around the Logger class,
so use the Logger directly. This also helps to reduce many warnings when
using the Eclipse IDE.

Add missing include statement. Error shows up with FairSoft nov20 and
FairRoot v18.4.2. Refs #2036.

- Add updated MC point matches to output of CbmEventClusterizer
- Add support for new "calibrated" vectors in MatchRecoToMc (no support for TClonesArray!)
- Adapt vectors/arrays names in FairRoot manager
- Adapt consummer classes to new names

Based on commit b1962019 by n.herrmann

Administrator authored 4 years ago and Florian Uhlig committed 4 years ago

On macosx one needs a openmp installation which does not come with the system,
at least the omp.h header file is missing. brew has a package libomp which
installs the header and a library. Both are found by the cmake module
FindOpenMP and properly used during the compilation and linking.
In themoment there is still a runtime error when executing the L1 tracking
with OpenMP support on macosx, so switch off OpenMP for the time being.
The problem is investigated already. refs #2028.

Cleanup OpenMP handling in the build system. Move check for OpenMP to top
level. Remove OpenMP usage from all libs were it is not needed.