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

Add / as allowed character in the regex which checks the copyright statement.
This is needed since we have cases with two institutes and two cities in the
copyright statement which are separated by a /.

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

Sergey Gorbunov authored 3 years ago and Volker Friese committed 3 years ago

remove resets of the random seed from the compiled code. We should keep the ability to reproduce all the results when a non-zero seed is set in the main macro

Corrects a variable typo which leads to some false positives

Allows passthrough of all files before trigger error code, to allow to see all files which don't obey header format.

Allows checking of individual files.

Eoin Clerkin authored 3 years ago and Florian Uhlig committed 3 years ago

Ensures that modified .cpp, .C and .h files follow basic licence header guidelines.

Check line 1 for a copyright declaration with correct numerial and character placements for a date followed by institute and city.
Check line 2 for SPDX licence identifier.
Check line 3 for author and committer information.

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

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

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

Flesnet generally uses C++17, but the fles_ipc classes can currently still be compiled as C++14.
This is an attempt to keep the cbmroot CI working with GCC 4.9.2.

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

Somehow (at least on some systems) the link order was such that the
first directory was $SIMPATH/lib. This was even the case when the order was
defined differently in the CMakeLists.txt. The problem isn't yet understood
completely.

This results in the problem that libVc.a was taken from $SIMPATH/lib
were it is installed by ROOT/FairSoft. As long as the version provided with
CbmRoot and the version provided by ROOT were the same the problem wasn't
seen. When updating our internal Vc version this results in a creash when
loading libryries due to a version mismatch between the library version and
the header file version.

Add tof digitization parameters for the mcbm 2021 beamtime.
CbmSoft/cbmroot_parameter!45

Administrator authored 3 years ago and Sergey Gorbunov committed 3 years ago

It was found that switching form FairSoft jun19p2 to nov20 or apr21 has a
dramatic influence on the performance of the KFParticleFinder. The runtime
for on of the tests increased by roughly a factor 16.
https://redmine.cbm.gsi.de/issues/2220
After long investigations it turned out that the compilation flags for the
package were inherited from the ROOT and the CbmRoot project which switched
of the performance flag "-O" when using FairSoft nov20 or apr21.
The merge request fixes the issue by defining the CMAKE_BUILD_TYPE explicitely
to be RelWithDebugInfo.
The merge request also updates the VC version from 1.4.1 to 1.4.2. refs #2220