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

Remove wrong license header drom several files which came in by using a
template. The code was developed by the respective persons, the license header
came in by using a skeleton.

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.

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

Comment unused variables and data members.
Fix printf formating issue which is different between macosx and Linux.

A static const class member in a cout statement results in an undefined symbol
when linking an executable using the class.
The solution is to do a static cast of the variable in the cout statement.

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

It was advised by the FariMQ expert to switch from a push-pull approach to a Rep-Req one.
This should improve the balance of TS delivery when they have non uniform size/processing time
and reduce the buffering in the Sample
- Add new sampler class CbmMQTsSamplerRepReq using the reply model
- Add example of a consummer device using the request model
- Add new monitor classes for T0 and TOF using the request model

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

[FLES IPC] In Multi-part subscriber, move the init out of the constructor to follow FairMQ states design
- Temporary patch to fles IPC external files
- Corresponding modifications in CbmMQTsaMultiSampler
- Corresponding modifications in CbmMcbm2018Source

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

- In CbmMqHistoServer, add a new method processing a multi-part message to combine Histo config, Canvas config and histo data
- Attach this new method to the histo data input channel
- Convert the T0 MQ monitor to this single channel usage
- Convert the Tof MQ monitor to this single channel usage

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

- In both, add InitHistograms method
- Call InitContainers and InitHistograms in Run and only on first message

Unify user names in the license header authors list.
Remove duplicate users in the authors list.
Cleanupo of some other inor issues.

Decision to not use doxygen for licence headers. Removes doxygen formatting and file tag.

Pulls some author and date information from previous headers, adds this to information from the svn and git repositories.
Inlude the standard copyright header to approximately 2000 files with institute of original committer as copyright holder.
Contributing authors from previous declaration and repository records.

Inserts the declaration on the 1st or 2nd line of scripts.

EMPTY LINE

Short license declarations for .sh files

Inserts the declaration on the 1st or 2nd line of scripts.

EMPTY LINE

Revert "Short license declarations for .sh files"

This reverts commit d3dd8608.

Uses SPDX identifier

Modifies declaration to use SPDX license identifier syntax

Corrects file

Empty line before hashbang removed

Files with detected previous copyright or authorship skipped.
Copyright claimed for the current or last CBMC institute of the original commiter to the SVN or GIT repositories.
File distributed under GNU public license version 3.0 declared.

Revert "Adds license header to .cxx, .cpp, .h, and .C files"

This reverts commit 94da15007a5d3a4e746cc751f1cf5956bfc48c5e.

Removes one blank line

Revert "Removes one blank line"

This reverts commit a8e5e236ea74822df8c88c7e785553c38ddf9c0b.

Preemptive empty lines removal

If first two lines of a file are empty, or have spaces and tabs only then remove the lines.

Revert "Preemptive empty lines removal"

This reverts commit da38bf5df03a1f5e34608ae035ddeffa729d9a78.

Add empty line

For stylistic reasons, an empty line is added after doxygen header.

Revert "Add empty line"

This reverts commit 30491934c3213e152f4b266cd34a3d65478259bc.

Puts space between includes.

Revert "Puts space between includes."

This reverts commit 468b70240d43541f51a64f7e00a11fabec09abe5.

Add multiple authors tag

Lists authors in order of their contribution

Revert "Add multiple authors tag"

This reverts commit f5d3e7486220d6adb1703b2a13e2246a221777ad.

Removes empty line on one file for clang format compliance.

Revert "Removes empty line on one file for clang format compliance."

This reverts commit fbac8bade3cefdc9b2c5b7c9be58ff2142a42976.

Small changes as per requested.

Add small (o) to original commiters username.

Adds running 10 percent instead of line count treshold for contributors.

Revert "Small changes as per requested."

This reverts commit 84af7b5ff4e28a03d180549b2fc999d325e3d269.

Long form of names

Word originator beside original committer of file.

Revert "Long form of names"

This reverts commit 759426e2b683c134bc6a3de0227c7e92918bbf6d.

Add space between comma seperated double affilliation institutes

The latest version of our formating rules is applied to all header and source
files as well as to the macros using clang-format 11.0.

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.

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

The current flesnet version contain all changes/patches which we did so far
when compiling CbmRoot.
The new flesnet version also needs un updated version of cppzmq.
refs. #2068, #2069

- Add threshold for non pulser digis in both algo, Task and device
- Make interval for spill threshold checks configurable in both algo, Task and device
- Spill detection only when both threshold passed
- Add Monitor macro for mCBM 2021 using this functionality

Remove check for ZeroMQ.
ZeroMQ isn't needed any longer.

Fix installation of externals when using Spack.
Pass extra flags flags to NicaFemto configuration if needed.
Patch ipc source code. Add missing include file.
Cleanup .gitignore in externals.

Fix linking of CbmGlobalTraking library.
Add missing link directory. In normal installation Vc probably was found in the
ROOT installation. With Spack ROOT is build without builtin Vc.

Use correct syntax for function call.
With the latest FairMQ version the old backward compatibility layer was
removed. Use now the new syntax to change the state.

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

Cleanup filtering of compiler warnings for Nightly tests

Remove unused variables, parameters and data members.
Remove move statement to allow copy ellision.
Use unsingned int in loop when comparing with size function of containers.
Initialize all data members.

Fix warning from rootcling.
With newer root versions the parameter -c isn't supported any longer,
so remove it.

First step to include code currently stored for historical reasons in the
fles directory, in the core or reco libraries of the respective detectors.
Adapt build system accordingly.

Pierre-Alain Loizeau authored 4 years ago and Volker Friese committed 4 years ago

Fixes in mCBM 2020 Algos and MQ examples:
- Remove JSROOT commands fore reset/save histos in MQ histos server and devices, as not working (need cling context!).
- Add/extend support for T0 channels remapping + related plots.
- Fix TOF pulser algo (logic was broken before and chain resetting the output plots).
Add macro folder + MQ monitor scripts for TOF and T0.